8000 Reconcile tools versions by aradhalevy · Pull Request #1943 · fybrik/fybrik · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Reconcile tools versions #1943

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
Merged
merged 9 commits into from
Feb 2, 2023
Merged

Reconcile tools versions #1943

merged 9 commits into from
Feb 2, 2023

Conversation

aradhalevy
Copy link
Contributor

reconciled the requirements to a single file, requirements.env, that used to be requirements.sh, and updated the places it used to be requirements.sh.

also updated kind version to 0.17.0

/closes #1906

note: the set -o nounset command that used to be in requirements.sh in unnecessary, as it is called in common.sh and works as expected

Signed-off-by: Arad Halevy arad.halevy@ibm.com

Arad Halevy added 2 commits January 24, 2023 14:05
reconciled the requirements to a single file
be47425 
Signed-off-by: Arad Halevy <arad.halevy@ibm.com>
update kube_version and changed requirements.sh to .env
0cb406c 
Signed-off-by: Arad Halevy <arad.halevy@ibm.com>
@aradhalevy aradhalevy self-assigned this Jan 24, 2023
@aradhalevy aradhalevy marked this pull request as ready for review January 30, 2023 07:52
@aradhalevy aradhalevy requested a review from roytman January 30, 2023 07:53
roytman
roytman reviewed Jan 30, 2023
View reviewed changes
@aradhalevy aradhalevy added the dependencies Pull requests that update a dependency file label Jan 31, 2023
Arad Halevy and others added 3 commits January 31, 2023 17:30
roytman
roytman reviewed Feb 1, 2023
View reviewed changes
changed according to review
576e2f9 
Signed-off-by: Arad Halevy <arad.halevy@ibm.com>
roytman
roytman reviewed Feb 2, 2023
View reviewed changes
@roytman roytman mentioned this pull request Feb 2, 2023
Merged
Arad Halevy and others added 2 commits February 2, 2023 21:13
@roytman roytman merged commit 3ed0bc2 into fybrik:master Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@roytman roytman roytman approved these changes

Assignees

@aradhalevy aradhalevy

Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Consolidate the third party required versions
2 participants
@aradhalevy @roytman
0