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

Add the upload artifact action #3167

Merged
merged 1 commit into from
Jun 25, 2020
Merged

Add the upload artifact action #3167

merged 1 commit into from
Jun 25, 2020

Conversation

paulbalandan
Copy link
Member

Description
Docs action still fails because it cannot cp the user guide folder. This adds the actions/upload-artifact. This is also referenced in ammaraskar/sphinx-action-test default workflow.

Checklist:

  • Securely signed commits
  • Component(s) with PHPdocs
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@lonnieezell
Copy link
Member

Awesome, thanks. I hadn't had a chance to get back to it yet.

@lonnieezell lonnieezell merged commit 275ee07 into codeigniter4:develop Jun 25, 2020
@paulbalandan paulbalandan deleted the docs-use-artifacts branch June 26, 2020 12:55
@samsonasik
Copy link
Member

The docs_nightly docs github action seems still failure in latest develop at https://github.com/codeigniter4/CodeIgniter4/runs/836152216


Run actions/upload-artifact@v2

##[warning]No files were found for the provided path: user_guide_src/_build/html/. No artifacts will be uploaded.
Run actions/upload-artifact@v2
##[warning]No files were found for the provided path: user_guide_src/_build/html/. No artifacts will be uploaded.
Commit documentation changes

##[error]Process completed with exit code 1.
Run git clone https://github.com/codeigniter4/CodeIgniter4.git --branch gh-pages --single-branch gh-pages
Cloning into 'gh-pages'...
cp: cannot stat 'user_guide_src/_build/html/*': No such file or directory
##[error]Process completed with exit code 1.

@paulbalandan could you re-check ?

@paulbalandan
Copy link
Member Author

That's strange. I'll dive into this later this evening. Thanks for the tip.

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