Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert Eclipse from Docbook to CommonMark #106532

Merged
merged 3 commits into from Feb 16, 2021
Merged

Conversation

ngiger
Copy link
Contributor

@ngiger ngiger commented Dec 10, 2020

Motivation for this change

Converted xml docs to md docs as per issue #105251

Things done

Converted xml to md and updated index.xml to reference it.

Copy link
Contributor

@jtojnar jtojnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also format the file using prettier as described in #105243?

Eclipse Platform with the popular Eclipse Color Theme plugin and also
allow Eclipse to use more RAM. You could then add

```ShellSession
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nix code.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry. I am no markdown specialist.

Should I replace ShellSession by nix?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. And same for the other Nix code.

are available for installation using `eclipseWithPlugins` by running

```ShellSession
$ nix-env -f '<nixpkgs>' -qaP -A eclipses.plugins --description
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be indented.

@ngiger
Copy link
Contributor Author

ngiger commented Dec 22, 2020

What do I have to do to get this pull request merged?

@jtojnar
Copy link
Contributor

jtojnar commented Dec 22, 2020

The review points still have not been resolved.

@ngiger ngiger force-pushed the eclipse-doc branch 2 times, most recently from 7e0ddcf to ebe1ea7 Compare December 25, 2020 22:07
@taku0
Copy link
Contributor

taku0 commented Feb 13, 2021

Please run the following command to run prettier:

nix-shell -I nixpkgs=channel:nixos-unstable -p nodePackages.prettier --run 'prettier --prose-wrap never --write doc/builders/packages/eclipse.section.md'

@jtojnar jtojnar merged commit 7656100 into NixOS:master Feb 16, 2021
@jtojnar
Copy link
Contributor

jtojnar commented Feb 16, 2021

Thanks.

@ngiger ngiger deleted the eclipse-doc branch March 12, 2022 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants