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

Move es6-components.js back into application.js #4216

Merged
merged 1 commit into from
Sep 10, 2024
Merged

Conversation

AshGDS
Copy link
Contributor

@AshGDS AshGDS commented Sep 2, 2024

What / Why

@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4216 September 2, 2024 14:50 Inactive
@AshGDS AshGDS self-assigned this Sep 6, 2024
@AshGDS AshGDS marked this pull request as ready for review September 6, 2024 08:58
@AshGDS AshGDS requested a review from MartinJJones September 6, 2024 09:47
Copy link
Contributor

@MartinJJones MartinJJones left a comment

Choose a reason for hiding this comment

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

Nice work Ash, the changes look good to me.

Although it wouldn't break anything, please can we also remove the reference to the es6-components file in https://github.com/alphagov/frontend/blob/main/.nycrc#L14

@AshGDS AshGDS force-pushed the type-module-cleanup branch from fa8485c to 9971659 Compare September 9, 2024 13:13
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4216 September 9, 2024 13:14 Inactive
@AshGDS
Copy link
Contributor Author

AshGDS commented Sep 9, 2024

Thanks @MartinJJones - have done that now, so should be ready for another review 👍

@AshGDS AshGDS merged commit c6afef1 into main Sep 10, 2024
12 checks passed
@AshGDS AshGDS deleted the type-module-cleanup branch September 10, 2024 11:19
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