Skip to content

Conversation

@larsoner
Copy link
Member

@larsoner larsoner commented Nov 2, 2021

Should fix https://app.circleci.com/pipelines/github/mne-tools/mne-python/11169. @marsipu ignore errors on Azure and CircleCI until this is merged and you can rebase or merge with upstream/main

@larsoner larsoner changed the title CI: Fix CircleCI and Azure [skip actions] CI: Fix CircleCI and Azure Nov 2, 2021
@larsoner larsoner merged commit f873c3f into mne-tools:main Nov 2, 2021
@larsoner larsoner deleted the circle branch November 2, 2021 14:32
$ cd $INSTALL_LOCATION
$ git clone https://github.com/$GITHUB_USERNAME/mne-python.git

.. sidebar:: Remote URLs in git
Copy link
Member Author

Choose a reason for hiding this comment

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

@drammock I got rid of this since it's not really relevant anymore. Feel free to open another PR if you think there are bits worth keeping

Copy link
Member

Choose a reason for hiding this comment

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

@larsoner what was the problem that was fixed here? The link to CircleCI in the PR description is 404 now.

Copy link
Member Author

Choose a reason for hiding this comment

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

git:// isn't supported anymore, you have to use https://

Copy link
Member

Choose a reason for hiding this comment

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

Ahh, ok. Sorry, I had only looked at this changed bit of docs, and was confused by the PR title. Now I see the other changes, it all makes sense.

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