Skip to content

USCIS: Export other artifacts#4461

Merged
thisisdano merged 2 commits into
uswds:developfrom
tyduptyler13:patch-1
Jan 18, 2022
Merged

USCIS: Export other artifacts#4461
thisisdano merged 2 commits into
uswds:developfrom
tyduptyler13:patch-1

Conversation

@tyduptyler13

Copy link
Copy Markdown
Contributor

Webpack 5 enforces correct export rules and thus prevents importing things like uswds.css.

Description

Upgrading to webpack 5 in our project caused our import of the css to break. The proposed change allows us to continue using this project without a fork/hack/or holding back a webpack upgrade.

Additional information

I don't have the webpack change on hand, but if I run across it again i'll update this ticket.

  • Follow the 18F Front End Coding Style Guide and Accessibility Guide.
  • Run npm test and make sure the tests for the files you have changed have passed.
  • Run your code through HTML_CodeSniffer and make sure it’s error free.
  • Title your pull request using this format: [Website] - [UI component]: Brief statement describing what this pull request solves.

Webpack 5 enforces correct export rules and thus prevents importing things like uswds.css.
@thisisdano

Copy link
Copy Markdown
Contributor

This looks good to me. @mejiaj can you confirm?

@thisisdano thisisdano merged commit 4da76bc into uswds:develop Jan 18, 2022
@thisisdano thisisdano mentioned this pull request Jan 19, 2022
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