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 errors on AppVeyor by updating pip packages #2141

Closed
wants to merge 8 commits into from

Conversation

itchyny
Copy link
Contributor

@itchyny itchyny commented Jun 12, 2020

This pull request fixes the error on AppVeyor by updating the pip packages.

@itchyny itchyny changed the title Update pip packages 20200612 Update pip packages Jun 12, 2020
@coveralls
Copy link

coveralls commented Jun 12, 2020

Coverage Status

Coverage remained the same at 84.134% when pulling 8c522a2 on itchyny:update-pip-packages-20200612 into d18b2d0 on stedolan:master.

@itchyny
Copy link
Contributor Author

itchyny commented Jun 12, 2020

There's no difference in output of build_mantests.py, build_manpage.py. The difference of build_website.py is as follows. This contains a small fix against the incorrect em tag at https://github.com/stedolan/jq/blob/76c044f301c3d33b122974e405a80f2b0775dce7/manual/v1.6/index.html#L5499-L5503.

diff -r output/manual/v1.5/index.html output2/manual/v1.5/index.html
4910,4913c4910,4915
< <p>where:
< <em> STRING, REGEX and FLAGS are jq strings and subject to jq string interpolation;
< </em> REGEX, after string interpolation, should be a valid PCRE regex;
< * FILTER is one of <code>test</code>, <code>match</code>, or <code>capture</code>, as described below.</p>
---
> <p>where:</p>
> <ul>
> <li>STRING, REGEX and FLAGS are jq strings and subject to jq string interpolation;</li>
> <li>REGEX, after string interpolation, should be a valid PCRE regex;</li>
> <li>FILTER is one of <code>test</code>, <code>match</code>, or <code>capture</code>, as described below.</li>
> </ul>
diff -r output/manual/v1.6/index.html output2/manual/v1.6/index.html
5499,5502c5499,5504
< <p>where:
< <em> STRING, REGEX and FLAGS are jq strings and subject to jq string interpolation;
< </em> REGEX, after string interpolation, should be a valid PCRE regex;
< * FILTER is one of <code>test</code>, <code>match</code>, or <code>capture</code>, as described below.</p>
---
> <p>where:</p>
> <ul>
> <li>STRING, REGEX and FLAGS are jq strings and subject to jq string interpolation;</li>
> <li>REGEX, after string interpolation, should be a valid PCRE regex;</li>
> <li>FILTER is one of <code>test</code>, <code>match</code>, or <code>capture</code>, as described below.</li>
> </ul>

@itchyny itchyny changed the title Update pip packages Fix errors on AppVeyor by updating pip packages Jun 12, 2020
@itchyny itchyny force-pushed the update-pip-packages-20200612 branch from 5a18712 to 5ae7966 Compare October 8, 2020 06:12
@itchyny
Copy link
Contributor Author

itchyny commented Oct 8, 2020

In this branch the AppVeyor CI staill fails but merging with #2193 should fix. Please confirm https://ci.appveyor.com/project/itchyny/jq/builds/35632771. Please consider including, the CI has been failing for months.

@itchyny itchyny force-pushed the update-pip-packages-20200612 branch from 5ae7966 to be6856c Compare May 2, 2021 03:54
@itchyny itchyny marked this pull request as draft May 2, 2021 04:05
@itchyny itchyny force-pushed the update-pip-packages-20200612 branch 3 times, most recently from 246c98f to 76157ef Compare May 2, 2021 04:22
@itchyny itchyny marked this pull request as ready for review May 2, 2021 04:34
@itchyny
Copy link
Contributor Author

itchyny commented May 2, 2021

Aha, AppVeyor CI is now back to green! The problem is that I am not familiar with pacman/Windows so not sure whether my fix is the correct way 🤔 . Are there anyone familiar with pacman/Windows?

@itchyny itchyny force-pushed the update-pip-packages-20200612 branch 3 times, most recently from fe44cb1 to 1ae3a64 Compare October 12, 2021 01:50
@itchyny
Copy link
Contributor Author

itchyny commented Oct 12, 2021

After unmaintained for a few months, now this branch gets failure again. I'm not familiar with pacman packages, get tired of tackling the problem. Is there anyone kindly look into the problem?

@itchyny
Copy link
Contributor Author

itchyny commented Apr 29, 2022

Closing due to conflict and inactivity. Hope someone familiar with Windows can fix the problem.

@itchyny itchyny closed this Apr 29, 2022
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