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

Deploy Docs to CliMA/ParameterEstimoceanDocumentation repo #277

Merged
merged 20 commits into from Jun 14, 2022

Conversation

navidcy
Copy link
Collaborator

@navidcy navidcy commented Jun 9, 2022

Closes #231

@navidcy navidcy added 📔 documentation Improvements or additions to documentation 📦 package labels Jun 9, 2022
docs/make.jl Outdated Show resolved Hide resolved
@glwagner
Copy link
Member

glwagner commented Jun 9, 2022

Looks like we have to make progress with the documentation issue first in order to merge more PRs?

@navidcy
Copy link
Collaborator Author

navidcy commented Jun 9, 2022

Looks like we have to make progress with the documentation issue first in order to merge more PRs?

No. The docs are not building because of incremental compilation errors (I think). But it's good to move the docs away from the main repo anyways.

@navidcy
Copy link
Collaborator Author

navidcy commented Jun 9, 2022

Something is actually going on with the docs... I'm puzzled...

@glwagner
Copy link
Member

Something is actually going on with the docs... I'm puzzled...

Should we try moving to 1.7?

@navidcy
Copy link
Collaborator Author

navidcy commented Jun 10, 2022 via email

@navidcy
Copy link
Collaborator Author

navidcy commented Jun 11, 2022

I’ll try!

I actually have trouble with Oceananigans + MPI locally on Julia v1.7.... aaaaarg!

@glwagner
Copy link
Member

Problem with documenter keys?

remote: Permission to CliMA/ParameterEstimoceanDocumentation.git denied to github-actions[bot].
fatal: unable to access 'https://github.com/CliMA/ParameterEstimoceanDocumentation.git/': The requested URL returned error: 403
┌ Error: Failed to push:
│   exception =
│    failed process: Process(`gitpush-fqupstreamHEAD:gh-pages`, ProcessExited(128)) [128]
│ 
│    Stacktrace:
│      [1] pipeline_error

@navidcy
Copy link
Collaborator Author

navidcy commented Jun 12, 2022

Perhaps! At least this is different than before :)

@navidcy navidcy merged commit 514e868 into main Jun 14, 2022
@navidcy navidcy deleted the ncc/move-docs-to-own-repo branch June 14, 2022 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📔 documentation Improvements or additions to documentation 📦 package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

We need a separate docs repository
2 participants