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 #64

Merged
merged 11 commits into from
Mar 1, 2024
Merged

Deploy #64

merged 11 commits into from
Mar 1, 2024

Conversation

mustafa-tariqk
Copy link
Owner

@mustafa-tariqk mustafa-tariqk commented Mar 1, 2024

What does this PR do?

Created a deploy action that runs whenever something is pushed to main. Deploys to digital ocean isntance defined in repository secrets.

Github Issue Number

Resolves: #44

Relates to:

Author checklist

If any of these points have yet to be satisfied, make sure that you set the title to the format DRAFT #issue-num Title

  • The title is short and descriptive of the PR. Must start with the Github Issue Number (format: #issue-num Title).
  • The description follows proper cl description practices and mentions related Github Issues (make sure this is the first thing you mention).
  • Branch has merged in the latest version of main
  • Linting has occured, as per the project linting config
  • All changed functions have proper docstring/wiki updates (front-end team) to describe what they do and how to use them.
  • Add yourself as the assignee.
  • Add reviewer a reviewer and let them know on discord.
  • Ensure that all relevant ticket has been linked to the PR

Reviewer checklist

  • Relevant issue is mentioned in description
  • Code solves the issue
  • Code follows the specification
  • Code is the best solution for the issue
  • Branch is ahead of main
  • Ensures the fix/feature works locally

Copy link
Collaborator

@BasicallyOk BasicallyOk left a comment

Choose a reason for hiding this comment

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

lgtm

@BasicallyOk BasicallyOk merged commit ba72d23 into main Mar 1, 2024
1 check passed
@mustafa-tariqk mustafa-tariqk deleted the deploy branch March 2, 2024 00:58
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.

Deployment
2 participants