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
[Vue] add test to make sure umd module files are production builds #18076
Conversation
@diosmosis would it make sense to also have a test that checks that the built vue files are up to date? Wondering if it otherwise could happen that someone changes a vue component but forgets to update or push the umd files. 🤔 |
@sgiehl That sounds like a good idea... I guess it could be done by checking if the diff w/ 4.x-dev has changes to Alternatively we could modify the travis-scripts to run the build command (after doing an npm install) and checking that nothing was modified. What do you think? |
Not sure if adding that to travis is a good option. Personally I would prefer moving such stuff to a github action. Similar to the PHPCS the action could simply run building the vue js files and check if there would be any changes available. That way we could even enforce the check and a PR couldn't be merged anymore if the files aren't up to date. |
@sgiehl ok, i'll look into using a github action. |
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
@diosmosis can we merge this already or do we need to wait for the github action? |
@tsteur the github action will prevent more issues so I'll still look into that. |
Closing in favor of #18080 which will rebuild vue files automatically. |
Description:
To catch when someone pushes a development build of a UMD module (as I did in the alert PR).
Review