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

remove travis, update setup-miniconda #401

Merged
merged 6 commits into from
Nov 11, 2020

Conversation

bollwyvl
Copy link
Collaborator

@bollwyvl bollwyvl commented Nov 6, 2020

References

Code changes

User-facing changes

  • removes travis badges from readme

Backwards-incompatible changes

N/A

Chores

  • linted
  • tested
  • documented
  • changelog entry

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Nov 6, 2020

I'd like to add py39, but it looks like there are a few remaining gotchas in the solve. I will investigate further, but we'll want to get this through to squash the warnings...

@bollwyvl bollwyvl changed the title [wip] remove travis, update setup-miniconda remove travis, update setup-miniconda Nov 9, 2020
@bollwyvl bollwyvl requested a review from krassowski November 9, 2020 03:20
@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Nov 9, 2020

Still can't figure out the py39 thing, or how to get conda to stop complaining, but have squashed a fair number of warnings... this is certainly a step forward!

@bollwyvl
Copy link
Collaborator Author

Thanks for the check. I really wish we could avoid the

Warning: 'defaults' already in 'channels' list, moving to the top

Junk, otherwise we'd be coming up clean, and the warnings list would be empty on a good run, rather than having to muck about...

@bollwyvl
Copy link
Collaborator Author

I think i may have gotten it down to a single error per job, which is... better...

@bollwyvl
Copy link
Collaborator Author

Nope, not really. force pushed back to the approved commit... guess it's going to fire again, but have no other stuff planned.

@bollwyvl
Copy link
Collaborator Author

Hm, master looks on fire. Haven't dug into it yet, hopefully will be able to resolve on #406... might not be tonight...

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