Skip to content

GitHub actions - #103

Open
bjorkypie wants to merge 30 commits into
mainfrom
github-actions
Open

GitHub actions#103
bjorkypie wants to merge 30 commits into
mainfrom
github-actions

Conversation

@bjorkypie

Copy link
Copy Markdown
Collaborator

Will the staging environment need to be set up once we have heroku? Everything up to that point is ready for review.

Issue #10 setting up Github Actions

@bjorkypie bjorkypie linked an issue Oct 24, 2021 that may be closed by this pull request
8 tasks
@bjorkypie
bjorkypie marked this pull request as ready for review October 24, 2021 00:47

@indifferentghost indifferentghost left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, but it'd be easier to just do the stuff on merge, more will come to this when I get heroku setup (sorry it's taking so long).

Comment thread .gitignore
Comment thread .github/workflows/merge-node.js.yml Outdated
Comment thread .github/workflows/merge-node.js.yml Outdated
Comment thread .github/workflows/merge-node.js.yml Outdated
- name: Install dependencies
run: yarn install --frozen-lockfile
- name: Build client package
run: CI='' yarn build --if-present

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I assume that the CI is used to fix some error?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

https://github.com/100devs/course-tracker/runs/4196173114?check_suite_focus=true

getting errors for warnings because CI=true when this was not set to empty string.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is the warnings we want to ensure aren't getting into the codebase. This should be a new issue.

@relspeedwagon relspeedwagon Nov 17, 2021

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Removed the CI line from both workflow files (had removed from one only in prev commit to test).

Warnings clean-up is a new issue

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.

Task - Setup GitHub Actions

4 participants