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

Release/0.38.0 #885

Merged
merged 7 commits into from
Aug 10, 2023
Merged

Release/0.38.0 #885

merged 7 commits into from
Aug 10, 2023

Conversation

alexanderleegs
Copy link
Contributor

@alexanderleegs alexanderleegs commented Aug 8, 2023

What's Changed

New environment variables:

  • WHITELISTED_GIT_SERVICE_REPOS : A comma-separated list of repositories that will use the GitFileSystemService instead of calling GitHub directly.

Full Changelog: v0.37.0...v0.38.0

seaerchin and others added 7 commits August 3, 2023 18:48
release(0.37.0): merge to develop
Snyk has created this PR to upgrade axios from 0.25.0 to 0.27.2.

See this package in npm:
https://www.npmjs.com/package/axios

See this project in Snyk:
https://app.snyk.io/org/isomer/project/676b9e26-cebf-4964-b7b3-d9843e3339ff?utm_source=github&utm_medium=referral&page=upgrade-pr

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
* Fix GitFile type to match actual output

* Add new configuration variable WHITELISTED_GIT_SERVICE_REPOS

* Introduce RepoService as drop-in replacement

* Utilise RepoService as the drop-in replacement

* Allow configuration variables to be tested

* Only checkout if needed and recover from known pull error

* Ensure we are on the right repo first

* Update unit tests to factor in ensureCorrectBranch

* Add unit testing for ensureCorrectBranch

* Catch concurrency issues with creating files

* Add handling for another type of pull error

* Add a git pull before listing directory contents

* Change location of whitelistRepos config

* Use more appropriate name ggsWhitelistedRepos

* Fix failing unit test after addition of git pull

* Return NotFoundError if ENOENT is received

* Include one more rare concurrency error

* Remove unnecessary never type

* Remove eager pulling and introduce commit and push

* Remove extra mocks after pulling is removed

* Add unit tests for push and commit functions
@alexanderleegs alexanderleegs marked this pull request as ready for review August 8, 2023 14:20
@alexanderleegs alexanderleegs mentioned this pull request Aug 8, 2023
@mergify
Copy link

mergify bot commented Aug 8, 2023

⚠️ The sha of the head commit of this PR conflicts with #886. Mergify cannot evaluate rules on this PR. ⚠️

@alexanderleegs alexanderleegs merged commit 316edcc into master Aug 10, 2023
@mergify mergify bot deleted the release/0.38.0 branch August 10, 2023 06:37
@harishv7 harishv7 restored the release/0.38.0 branch August 15, 2023 08:35
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.

5 participants