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

PDT24-18 | Github Action Integration #8

Merged
merged 3 commits into from
Dec 19, 2024
Merged

PDT24-18 | Github Action Integration #8

merged 3 commits into from
Dec 19, 2024

Conversation

noxiousghost
Copy link
Collaborator

Tasks

Changes

  • Created a pull request template file with basic pull request formatting
  • Created a github actions workflow file to check for code linting on every push

@arjandhakal
Copy link
Collaborator

@noxiousghost good job!!

Comment on lines +8 to +19
- name: Check out Git repository
uses: actions/checkout@v3
- name: Set up Node.js
uses: actions/setup-node@v3
with:
node-version: 20
cache: yarn
cache-dependency-path: './yarn.lock'
- name: Install dependencies
run: yarn install
- name: Run linters
run: yarn lint && yarn format
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not a big issue but you can separate each step with an empty line so it's easier to read jobs with large number of steps :)

Copy link
Collaborator

@rrojan rrojan left a comment

Choose a reason for hiding this comment

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

LGTM 🏆

@noxiousghost noxiousghost merged commit e48dc97 into main Dec 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants