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

Removed usage of fs_str from vistir #5059

Closed
wants to merge 1 commit into from
Closed

Conversation

oz123
Copy link
Contributor

@oz123 oz123 commented Apr 19, 2022

As discussed, I am suggesting to remove fs_str.

@oz123 oz123 requested a review from matteius April 19, 2022 15:43
This function was all about compatability of py2-py3.3 versions.
Later versions don't need it.
@oz123 oz123 force-pushed the remove-vistir-fs_str branch 4 times, most recently from a593b0b to 535359d Compare April 19, 2022 20:39
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
@oz123
Copy link
Contributor Author

oz123 commented Apr 19, 2022

OK, The latest commits don't have much to do with the title, but here is what they do:
We migrate from a CI pipeline the looks like this:

image

To this:

image

There is an explicit step for linting, only if it passes the tests will run. If these pass, the vendoring will trigger, and once this is done we build a package with update vendoring code.

@oz123 oz123 requested a review from matteius April 19, 2022 21:00
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
@oz123 oz123 force-pushed the remove-vistir-fs_str branch from 4fb21d4 to 68cc0a7 Compare April 20, 2022 09:22
@oz123 oz123 closed this Apr 20, 2022
This was referenced Apr 20, 2022
@oz123 oz123 deleted the remove-vistir-fs_str branch July 19, 2022 18:13
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.

2 participants