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

Prepping for 1.2.4 release #149

Merged
merged 3 commits into from
Jul 16, 2020
Merged

Prepping for 1.2.4 release #149

merged 3 commits into from
Jul 16, 2020

Conversation

jlaura
Copy link
Collaborator

@jlaura jlaura commented Jul 15, 2020

No description provided.

@ladoramkershner
Copy link
Contributor

Just waiting for the tests to pass

appveyor.yml Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Jul 15, 2020

Coverage Status

Coverage remained the same at 50.623% when pulling 30c1444 on jlaura:master into ec59585 on USGS-Astrogeology:master.

@jlaura
Copy link
Collaborator Author

jlaura commented Jul 15, 2020

I also do not understand what is going on with appveyor. I saw the same docs, so assumed this should be working...Reran the job once to make sure this wasn't just an intermittent thing.

I can roll back 3.8 support on win or this can get shelved for a bit until I have more time. Thoughts?

@ladoramkershner
Copy link
Contributor

Is this release just to get out the most recent plio change (PR #148)?

@jlaura
Copy link
Collaborator Author

jlaura commented Jul 15, 2020

@ladoramkershner Yup. Just to get the release out. So, if it sits for a bit it isn't a big deal as along as you are good rolling. Typing that, maybe the best thing to do is for me to open an issue and get to that in the future.

@jlaura
Copy link
Collaborator Author

jlaura commented Jul 15, 2020

@ladoramkershner I'm doing a test now. Based on how the file is structured, it looks like we can specify 3-x64 to get miniconda3 installed. Then we set up an environment with the PYTHON_VERSION version installed, so any 3 should work. I'm testing on the 3.8.

@jlaura
Copy link
Collaborator Author

jlaura commented Jul 16, 2020

@ladoramkershner @jessemapel This was fixed using the approach above. Here is the line in the 3.8 where the correct python is installed: https://ci.appveyor.com/project/jlaura/plio/build/job/uutxlso6tvtx8tu0#L118

Tests are using 3.8 like we want: https://ci.appveyor.com/project/jlaura/plio/build/job/uutxlso6tvtx8tu0#L882

I won't know about the Conda build and release until this is merged because we don't conda build a PR.

@jessemapel jessemapel merged commit deb2cf5 into DOI-USGS:master Jul 16, 2020
Kelvinrr pushed a commit to Kelvinrr/plio that referenced this pull request May 26, 2022
* Prepping for 1.2.4 release

* Update appveyor.yml

* Update appveyor.yml
ladoramkershner pushed a commit to ladoramkershner/plio that referenced this pull request Jun 7, 2022
* Prepping for 1.2.4 release

* Update appveyor.yml

* Update appveyor.yml
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.

4 participants