Skip to content

TST: Sync Actions between 4.2.x and master branch - #11122

Merged
pllim merged 3 commits into
astropy:v4.2.xfrom
pllim:last-actions-hero-2
Dec 4, 2020
Merged

pllim merged 3 commits into
astropy:v4.2.xfrom
pllim:last-actions-hero-2

Conversation

@pllim

@pllim pllim commented Dec 2, 2020 •

Copy link
Copy Markdown
Member

Description

This pull request is to address the 4.2.x release branch for #11050 .

Expected failures:

@pllim

pllim commented Dec 2, 2020

Copy link
Copy Markdown
Member Author

Not sure why it says Running tests with Astropy version 4.3.dev4+g4a54fc7d4. Maybe setuptools_scm is just confused. 🤷

@saimn

saimn commented Dec 2, 2020

Copy link
Copy Markdown
Contributor

setuptools_scm guesses the next version number from the previous tag, so the version after 4.2 is 4.3, not 4.2.1

@pllim
pllim marked this pull request as ready for review December 2, 2020 22:03
@pllim
pllim requested review from astrofrog and saimn December 2, 2020 22:03
Comment thread .github/workflows/ci_cron_weekly.yml Resolved
@Cadair

Cadair commented Dec 2, 2020

Copy link
Copy Markdown
Member

setuptools_scm guesses the next version number from the previous tag, so the version after 4.2 is 4.3, not 4.2.1

one of many reasons one shouldn't drop digits from versions when they are 0...

- name: Bundling with pyinstaller
os: ubuntu-latest
python: 3.8
toxenv: pyinstaller

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe move this to the main workflow as well ? It seems useless to run it daily as there are not commits every day pushed to those release branches.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

All the cron files are actually push events now. I added comments to explain that I keep the files as-is for ease of backporting (in case we do a 4.2.x release).

The unused block has to be commented out.

TST: Remove 32-bit job from CircleCI.
@pllim

pllim commented Dec 3, 2020

Copy link
Copy Markdown
Member Author

I cancelled Actions for the commit to remove only .travis.yml. See the CI results for the commit before that one.

@saimn saimn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Maybe the azure failure deserves a new issue ?

@pllim

pllim commented Dec 4, 2020

Copy link
Copy Markdown
Member Author

I am pretty confident that Azure failure will go away once #11089 is merged, I just need someone to review it to make sure CI is defined correctly as per Azure syntax.

@pllim
pllim merged commit 5baaee8 into astropy:v4.2.x Dec 4, 2020
@pllim
pllim deleted the last-actions-hero-2 branch December 4, 2020 13:36
@pllim

pllim commented Dec 4, 2020

Copy link
Copy Markdown
Member Author

Let's get this in then. Thanks for the review!

@mhvk

mhvk commented Dec 4, 2020

Copy link
Copy Markdown
Contributor

Just noticed that with this one we have exactly 10000 closed issues!

@pllim

pllim commented Dec 4, 2020

Copy link
Copy Markdown
Member Author

I also snatched #11111 . Pats self on back. 😹

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants