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

documentation: update individual extension readmes #8279

Merged
merged 1 commit into from
Aug 3, 2020

Conversation

vince-fugnitto
Copy link
Member

What it does

The following pull-request updates the individual extension readmes to refer to Eclipse Theia rather than Theia as per the Eclipse Foundation first-mention rule. Since the readmes are displayed on npm, they would require the name Eclipse as it is a first-mention.

Uses the formal name including appropriate marks, e.g. Eclipse Foo™, on the page title, first mention in the text, and on all prominent references to the project;

The pull-request also includes updates to the following extensions to use the template:

  • @theia/api-samples
  • @theia/ext-scripts

How to test

Verify the content of the documentation.

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto [email protected]

@vince-fugnitto vince-fugnitto added the documentation issues related to documentation label Jul 30, 2020
@vince-fugnitto vince-fugnitto self-assigned this Jul 30, 2020
Copy link
Contributor

@marcdumais-work marcdumais-work left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @vince-fugnitto . This brings down to just a couple the extensions that do not yet follow the README template: core and filesystem.

- the following commit updates individual extension readmes to include
`Eclipse Theia` instead of `Theia` following the foundation rules.

Signed-off-by: vince-fugnitto <[email protected]>
@vince-fugnitto vince-fugnitto merged commit fe24630 into master Aug 3, 2020
@vince-fugnitto vince-fugnitto deleted the vf/eclipse-name branch August 17, 2020 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation issues related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants