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

fix: ran package to bundle latest changes #709

Closed
wants to merge 2 commits into from

Conversation

jackie-linz
Copy link
Contributor

forgot to run package in previous PR (#707)

dist/** -diff linguist-generated=true
__tests__/resource/assets/archive-example-test-4.txt eol=lf
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@robinraju this is what you need to fix the test failure on Windows

BTW - is it normal that builds from forks fail?

I see you opened a PR too, maybe just pick this change into your branch?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

test fail because git clone on Windows would use CRLF whereas the file in the zip file has just LF

Copy link
Owner

Choose a reason for hiding this comment

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

Tests from forks fail usually because of a test I setup to verify downloading files from private repos.
This test requires a GH access token which I configured using secrets, and for security reasons, github doesn't share it with forks.

Copy link
Owner

Choose a reason for hiding this comment

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

I merged a PR to fix the test failure. The change just normalise the EOL character within tests.
The builds are passing now. Also I included the generated dist/index.js with that.
Thanks again for for sending a fix :)

@robinraju
Copy link
Owner

superseded by #710

@robinraju robinraju closed this Apr 9, 2024
@jackie-linz jackie-linz deleted the fix/unzip branch April 9, 2024 11:36
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