Skip to content

docs: update testing chapter with general test commands#18714

Merged
alan-agius4 merged 1 commit into
angular:masterfrom
Jefiozie:fix_testing_readme
Sep 8, 2020
Merged

docs: update testing chapter with general test commands#18714
alan-agius4 merged 1 commit into
angular:masterfrom
Jefiozie:fix_testing_readme

Conversation

@Jefiozie

@Jefiozie Jefiozie commented Sep 7, 2020

Copy link
Copy Markdown
Contributor

Updated the information around testing as discussed in #18710

Closes: #18710

Comment thread README.md Outdated

@alan-agius4 alan-agius4 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The change needs to be done in the readme template https://github.com/angular/angular-cli/blob/master/scripts/templates/readme.ejs

Afterwards you need to run yarn admin templates to generate the MD file.

@Jefiozie

Jefiozie commented Sep 7, 2020

Copy link
Copy Markdown
Contributor Author

Sorry, I didn't know this will apply these changes.

Comment thread README.md Outdated
Comment thread README.md
Comment thread scripts/templates/readme.ejs Outdated

@alan-agius4 alan-agius4 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for this.

A couple of small comments above.

@Jefiozie

Jefiozie commented Sep 8, 2020

Copy link
Copy Markdown
Contributor Author

I've made the requested changes let me know if I need to make more changes.

@Jefiozie Jefiozie requested a review from alan-agius4 September 8, 2020 07:15

@alan-agius4 alan-agius4 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release labels Sep 8, 2020
@alan-agius4 alan-agius4 merged commit b2f0f07 into angular:master Sep 8, 2020
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Oct 9, 2020
@Jefiozie Jefiozie deleted the fix_testing_readme branch October 27, 2024 08:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect/ missing contribution information in README

3 participants