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

Rework the readme a bit #680

Merged
merged 5 commits into from
Dec 7, 2022
Merged

Rework the readme a bit #680

merged 5 commits into from
Dec 7, 2022

Conversation

hoechenberger
Copy link
Member

@hoechenberger hoechenberger commented Dec 7, 2022

Before merging …

  • Changelog has been updated (docs/source/changes.md)

Closes #679

@hoechenberger hoechenberger changed the title Rework the readme a bit [ci skip] Rework the readme a bit Dec 7, 2022
@hoechenberger hoechenberger marked this pull request as ready for review December 7, 2022 10:12
@hoechenberger
Copy link
Member Author

@larsoner @agramfort Please merge if happy, thanks!

README.md Outdated Show resolved Hide resolved
@larsoner
Copy link
Member

larsoner commented Dec 7, 2022

Will merge after I make sure #679 is fixed over SSH. I already pushed what I think should fix things here

README.md Outdated Show resolved Hide resolved
@larsoner
Copy link
Member

larsoner commented Dec 7, 2022

Okay, docs are deploying for 1.0.1 and I updated instructions in this PR. Let's merge and see if dev deploys next...

@larsoner larsoner merged commit be01326 into main Dec 7, 2022
@larsoner larsoner deleted the readme branch December 7, 2022 15:51
@larsoner
Copy link
Member

larsoner commented Dec 7, 2022

Okay the latest dev push appears to have worked! 🎉

@hoechenberger
Copy link
Member Author

Super nice! Thanks Eric!

@larsoner
Copy link
Member

larsoner commented Dec 7, 2022

@hoechenberger worth cutting 1.0.2 so it looks better on PyPI, or no?

If so, I'll wait until mne-tools/mne-python#11359 is merged then cut the release here, that way we can make the stable website smaller

@hoechenberger
Copy link
Member Author

Yes sounds good.

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.

Doc deployment step failing
2 participants