Skip to content

Add a package.json file for a better javascript development experience #365

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

Merged
merged 3 commits into from
Nov 10, 2020

Conversation

gnikonorov
Copy link
Member

@gnikonorov gnikonorov commented Nov 7, 2020

Add a package.json file to simplify the experience of developing the JS portion of this plugin.

This also better sets us up to tackle #350 since we won't have to remember all the JS dependencies anymore, and can use more modern practices for JS development.

@gnikonorov gnikonorov added skip-changelog Can be missed from the changelog. Infrastructure Changes related to project infrastructure ( CI/CD, deploy mechanism, etc. ) labels Nov 7, 2020
@gnikonorov gnikonorov self-assigned this Nov 7, 2020
@gnikonorov
Copy link
Member Author

@ssbarnea I ended up removing all data from the package.json that was related to publishing, so now it only has what is necessary to set up the JavaScript environment for and run the tests. I think it is good for another look when you have a chance.

@gnikonorov
Copy link
Member Author

Bump @ssbarnea or @BeyondEvil

"grunt-contrib-qunit": "^4.0.0",
"phantomjs-prebuilt": "2.1.15"
},
"devDependencies": {},
Copy link
Member Author

Choose a reason for hiding this comment

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

Note that I intentionally left this in. I'd rather be explicit in listing that there are no dev dependencies than have people guessing.

Copy link
Contributor

@BeyondEvil BeyondEvil left a comment

Choose a reason for hiding this comment

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

LGTM! One small comment.

Comment on lines +78 to +87
1. Update your local master with the upstream master (``git pull --rebase upstream master``)
2. Create a new branch
3. Update ``CHANGES.rst`` with the new version, today's date, and all changes/new features
4. Update the ``version`` field in ``package.json`` with the new version
5. Commit and push the new branch and then create a new pull request
6. Wait for tests and reviews and then merge the branch
7. Once merged, update your local master again (``git pull --rebase upstream master``)
8. Tag the release with the new release version (``git tag v<new tag>``)
9. Push the tag (``git push upstream --tags``)
10. Done. Check `CI <https://github.com/pytest-dev/pytest-html/actions>`_ for release progress.
Copy link
Contributor

Choose a reason for hiding this comment

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

Here's a trick: You can change all the numbers to 1, and RST will renumber them. That way, if you have to insert something, it will only be a one-line change. :)

@ssbarnea ssbarnea merged commit 75b481e into pytest-dev:master Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Changes related to project infrastructure ( CI/CD, deploy mechanism, etc. ) skip-changelog Can be missed from the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants