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

Update warning text README #436

Merged
merged 2 commits into from
Jan 18, 2018
Merged

Update warning text README #436

merged 2 commits into from
Jan 18, 2018

Conversation

36degrees
Copy link
Contributor

In #432 changes were made directly to README.md that should have been made in index.njk, mainly to update references to the component name, but also to change the 'description' to remove a reference to legal consequences.

This updates index.njk to include the change to the description and regenerates the readme with the correct whitespace and capitalisation as well as changes to the arguments table and to the macro example.

In [#432](#432) changes were made directly to README.md that should have been made in index.njk, mainly to update references to the component name, but also to change the 'description' to remove a reference to legal consequences.

This updates index.njk to include the change to the description and regenerates the readme with the correct whitespace and capitalisation as well as changes to the arguments table and to the macro example.
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-436 January 18, 2018 11:12 Inactive
Copy link
Contributor

@joelanman joelanman left a comment

Choose a reason for hiding this comment

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

looks good

@36degrees 36degrees merged commit 224961c into master Jan 18, 2018
@36degrees 36degrees deleted the update-warning-text-readme branch January 18, 2018 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants