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

fix: Overwrite incorrect sri-history entries #789

Merged
merged 1 commit into from
Mar 27, 2018
Merged

Conversation

isner
Copy link
Contributor

@isner isner commented Mar 20, 2018

I validated each of the sri-history entries manually and discovered that the following 3 versions have inaccurate entries:

  • 2.4.2
  • 2.6.1
  • 3.0.0-beta.1

I am personally responsible for adding the 2.4.2 and 2.6.1 entries. I'm fairly certain that I used the build/sri-update tool to generate those entries back when I added them, and today I generated the entries by npm iing axe-core and generating the hash using the sri-toolbox module. I don't see why those two methods would produce different results, so I'm baffled. I'm guessing human error.

Copy link
Contributor

@WilcoFiers WilcoFiers left a comment

Choose a reason for hiding this comment

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

Second thought, the 3.0.0 one is also wrong.

@isner
Copy link
Contributor Author

isner commented Mar 21, 2018

@WilcoFiers Right you are. I've fixed that one too.

@WilcoFiers WilcoFiers merged commit f5f55f3 into develop Mar 27, 2018
@WilcoFiers WilcoFiers deleted the sri-fixes branch March 27, 2018 08:49
@marcysutton
Copy link
Contributor

I wonder if it's missing an automated build step so npm build will run before npm run release gets fully executed. npm build gets triggered for npm publish because of a similar manual building requirement that tripped us up when releasing.

@WilcoFiers
Copy link
Contributor

@marcysutton They added a bunch of lifecycle hooks to standard-version as of 4.1.0. That should help with this. node build/sri-update should run in the postbump hook.

mrtnvh pushed a commit to mrtnvh/axe-core that referenced this pull request Nov 24, 2023
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