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

eJWST: remove disclaimer for release #2243

Merged
merged 2 commits into from
Dec 7, 2021

Conversation

jespinosaar
Copy link
Contributor

@jespinosaar jespinosaar commented Dec 7, 2021

Hi @bsipocz , @keflavich,

This pull request follows #2238, to remove the disclaimer for users, as it is not needed anymore (my fault, I forgot to remove it). As we agreed, release of v0.4.5 should be out by launch, but, at least from our side, you can start right after this pull request is merged to ensure we can solve any unrelated issue and, if they do not appear, everything will be ready even before launch. Everything is in place in this module.

Kind regards and again, many thanks for your help.

Javi

@jespinosaar jespinosaar mentioned this pull request Dec 7, 2021
@codecov
Copy link

codecov bot commented Dec 7, 2021

Codecov Report

Merging #2243 (9dce73c) into main (54ca5fe) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2243   +/-   ##
=======================================
  Coverage   62.38%   62.38%           
=======================================
  Files         131      131           
  Lines       16777    16777           
=======================================
  Hits        10466    10466           
  Misses       6311     6311           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 54ca5fe...9dce73c. Read the comment docs.

@jespinosaar
Copy link
Contributor Author

jespinosaar commented Dec 7, 2021

I also added the entry in the changelog, I think the check will be a success when you include the milestone.

@bsipocz bsipocz added this to the v0.4.5 milestone Dec 7, 2021
@bsipocz bsipocz merged commit 4f29262 into astropy:main Dec 7, 2021
@bsipocz
Copy link
Member

bsipocz commented Dec 7, 2021

Thanks @jespinosaar!

@jespinosaar jespinosaar deleted the ejwst_remove_disclaimer branch December 16, 2021 12:18
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.

2 participants