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

upgrade deps to latest versions #514

Merged
merged 1 commit into from
Dec 15, 2020
Merged

upgrade deps to latest versions #514

merged 1 commit into from
Dec 15, 2020

Conversation

gugu
Copy link
Contributor

@gugu gugu commented Dec 13, 2020

Description

Thank you for taking the time to create a PR to support this commuity-maintained project. We are all volunteers here, so features are only added by contributers like you. Please also be patient while the maintainers work to review your proposed changes for quality and effacy.

Please include a summary of the change and, where appropriate, which issue is addressed.

Your description should also include a comment about the use-case for this change, including what systems you may be interfacing with that require this change. This will help future users know how this library is being used in the wild and how well supported it is.

If the change involves the SAML specification, please include a link to the relavent part(s) of the SAML specificaiton. Doing so speeds up the review process and helps the maintainers make sure that this project remains spec-compliant. You might start here to find the part of the specificaiton that relates to this PR.

Please include tests. Doing so will ensure that the changes made in this PR are not undone or otherwise corrupted by future changes.

Checklist:

  • Issue Addressed: [ ]
  • Link to SAML spec: [ ]
  • Tests included? [ ]
  • Documentation updated? [ ]

@gugu gugu merged commit ed13176 into master Dec 15, 2020
@gugu gugu deleted the upgrade-deps branch December 15, 2020 10:42
@gugu gugu added the dependencies Pull requests that update a dependency file label Jan 7, 2021
@cjbarth cjbarth mentioned this pull request May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants