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.

Sign up for GitHub

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

Jump to bottom

Release v6.2.0 #276

Merged
merged 29 commits into from
May 17, 2024
Merged

Release v6.2.0 #276

merged 29 commits into from
May 17, 2024

Conversation

forrestfwilliams
Copy link
Contributor

No description provided.

dependabot bot and others added 25 commits February 28, 2024 03:18
@dependabot
Bump pypa/gh-action-pypi-publish from 1.8.11 to 1.8.12
3429a07 
Bumps [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish) from 1.8.11 to 1.8.12.
- [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases)
- [Commits](pypa/gh-action-pypi-publish@v1.8.11...v1.8.12)

---
updated-dependencies:
- dependency-name: pypa/gh-action-pypi-publish
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@jhkennedy
Drop verify step for PyPI distribution
861a676
@jtherrmann
Merge pull request #265 from ASFHyP3/jhkennedy-patch-1
7564709 
Drop verify step for PyPI distribution
@jtherrmann
Merge pull request #263 from ASFHyP3/dependabot/github_actions/pypa/g…
d054d3d 
…h-action-pypi-publish-1.8.12

Bump pypa/gh-action-pypi-publish from 1.8.11 to 1.8.12
@dependabot
Bump pypa/gh-action-pypi-publish from 1.8.12 to 1.8.14
8da85d5 
Bumps [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish) from 1.8.12 to 1.8.14.
- [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases)
- [Commits](pypa/gh-action-pypi-publish@v1.8.12...v1.8.14)

---
updated-dependencies:
- dependency-name: pypa/gh-action-pypi-publish
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@jhkennedy
Merge pull request #267 from ASFHyP3/dependabot/github_actions/pypa/g…
c574d91 
…h-action-pypi-publish-1.8.14

Bump pypa/gh-action-pypi-publish from 1.8.12 to 1.8.14
@jtherrmann
add priority
1712171
@jtherrmann
update tests
df8ef8c
@jtherrmann
changelog
0acf3a1
@asjohnston-asf
Update dependabot.yml to weekly
91e70bc
@asjohnston-asf
Merge pull request #273 from ASFHyP3/asjohnston-asf-patch-1
6b7b7a7 
Update dependabot.yml to weekly
@jacquelynsmale
first commit
3527ebc
@jacquelynsmale
add in check_app_status test
aecdf92
@AndrewPlayer3
fixed tests to work with _check_application_status
bf0befd
@AndrewPlayer3
flake8
2d83008
@AndrewPlayer3
added hyp3 instance as fixture
0984f6a
@AndrewPlayer3
flake8
24ec80a
@AndrewPlayer3
flake8
d77bfbe
@AndrewPlayer3
removed unused function definitions
2e324d0
@jacquelynsmale
update CHANGELOG and add url
cf1607d
@jacquelynsmale
add in exceptions
8d23ec6
@jacquelynsmale
fix logic and add test
1bc4d4e
@jacquelynsmale
fix blank space
4162490
@forrestfwilliams
Update src/hyp3_sdk/hyp3.py
55e2617
@forrestfwilliams
Merge pull request #275 from ASFHyP3/check-whitelist
33fb8f0 
Check user is whitelisted after authentication
@forrestfwilliams forrestfwilliams added the patch Bump the patch version number of this project label May 16, 2024
@forrestfwilliams forrestfwilliams requested a review from a team May 16, 2024 12:05
@github-actions GitHub Actions
Copy link

github-actions bot commented May 16, 2024 •
edited by jtherrmann
Loading

Developer checklist

  • Indicated the level of changes to this package by affixing one of these labels:
    • major -- Major changes to the API that may break current workflows
    • minor -- Minor changes to the API that do not break current workflows
    • patch -- Patches and bugfixes for the current version that do not break current workflows
    • bumpless -- Changes to documentation, CI/CD pipelines, etc. that don't affect the software's version
  • (If applicable) Updated the dependencies and indicated any downstream changes that are required
  • Added/updated documentation for these changes
  • Added/updated tests for these changes
  • Verified changes in test deployment and summarized results, e.g. in PR description or comments on the related issue(s)

Reviewer checklist

  • Have all dependencies been updated?
  • Is the level of changes labeled appropriately?
  • Are all the changes described appropriately in CHANGELOG.md?
  • Has the documentation been adequately updated?
  • Are the tests adequate?
  • Have the changes been verified in the test deployment?

@forrestfwilliams forrestfwilliams added minor Bump the minor version number of this project and removed patch Bump the patch version number of this project labels May 16, 2024
@forrestfwilliams forrestfwilliams changed the title Release v6.1.1 Release v6.2.0 May 16, 2024
jtherrmann
jtherrmann reviewed May 16, 2024
View reviewed changes
src/hyp3_sdk/hyp3.py Outdated Show resolved Hide resolved
jtherrmann
jtherrmann reviewed May 16, 2024
View reviewed changes
Copy link
Contributor

@jtherrmann jtherrmann left a comment

Choose a reason for hiding this comment

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

LGTM, aside from my suggestion and question.

@jtherrmann
Copy link
Contributor

Has someone tested these changes locally with a user with each of the four application statuses?

@jacquelynsmale
Copy link
Contributor

Has someone tested these changes locally with a user with each of the four application statuses?

I had just tested with the test cases. Do you have example accounts for each of the application statuses or a way to go in and change that status in the AWS Dynamo DB?

@jtherrmann
Copy link
Contributor

I get all of the expected warning messages when testing locally.

@jtherrmann jtherrmann merged commit c26e8fc into main May 17, 2024
13 checks passed
jtherrmann added a commit that referenced this pull request Jul 11, 2024
@jtherrmann
Revert unapproved user check from #276
5a9894d
@jtherrmann jtherrmann mentioned this pull request Jul 11, 2024
Closed
jtherrmann added a commit that referenced this pull request Jul 11, 2024
@jtherrmann
Revert unapproved user check from #276
72e6f76
@jtherrmann jtherrmann mentioned this pull request Jul 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jtherrmann jtherrmann jtherrmann approved these changes

Assignees
No one assigned
Labels
minor Bump the minor version number of this project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@forrestfwilliams @jtherrmann @jacquelynsmale @jhkennedy @asjohnston-asf @AndrewPlayer3