Skip to content

Conversation

@JohnGriffiths
Copy link
Collaborator

Several issues with the current docs site. Need to solve this docs building issue, even if just a temporary fix, as we are (hoping for!) lots of eyes on this site right now.

AFAIK this has to be an iterative process of commit, check action status, commit, etc. Interactive prompt within gh actions would be handy, I don't know how to do that yet.

@ErikBjare
Copy link
Collaborator

The resulting error is one of the things I fixed in #167 by removing version locks: https://github.com/NeuroTechX/eeg-notebooks/pull/167/files#diff-4d7c51b1efe9043e44439a949dfd92e5827321b34082903477fd04876edb7552L48-R54

@JohnGriffiths
Copy link
Collaborator Author

ok thanks, I can add those back in here.

the version specs seem to be important for some libs that break with new version of dependencies. but that's likely not the case for most of our dependencies, and indeed the converse is also an issue, as we have here.

@JohnGriffiths
Copy link
Collaborator Author

woop woop we have a pass on docs build.

I vote to merge. @ErikBjare you agree?

@JohnGriffiths JohnGriffiths merged commit 8ee4ffc into master Mar 28, 2022
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.

3 participants