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 v1.1.1 #292

Merged
merged 12 commits into from
Mar 14, 2024
Merged

Deploy v1.1.1 #292

merged 12 commits into from
Mar 14, 2024

Conversation

gycgabriel
Copy link
Collaborator

@gycgabriel gycgabriel commented Mar 14, 2024

Summary:

Type of change:

(Delete where appropriate)

  • 🌊 Deployment

Changes Made:

  • As seen in commit log below

Screenshots:

(If applicable, provide screenshots or GIFs to visually demonstrate the changes.)

Proposed Commit Message:

Deploy v1.1.1

Checklist:

  • I have tested my changes thoroughly.
  • I have created tests for any new code files created in this PR or provided a link to a issue/PR that addresses this.
  • I have added or modified code comments to improve code readability where necessary.
  • I have updated the project's documentation as necessary.

Arif-Khalid and others added 12 commits March 4, 2024 12:02
Making releases is done via a manual call
to Version Deployment workflow

This can be further streamlined by deployment
on push to a deploy branch

Let's change workflow behavior to run on push
to a deploy branch
Title filter is handled outside of filters service

This violates the responsibility of the filters service

Let's refactor the search filter to use the filters service
Npm run test will break if the import statement 
for zone-testing is not at the top of test.ts file. 

Let's reorder the import statement and disable
tslint rule for test.ts file.
The method in issue sorter has been updated.

Let's refactor the test cases to match the changes.
The test cases for the user service are outdated. 

Let's refactor test cases for user service.
The test cases for the label filter bar component are outdated. 

Let's refactor test cases for label filter bar component
Refactor and add test cases for phase service

Phase service has been updated. 

Let's refactor and add test cases to match the changes.
pre-push hook that runs `npm run test` automatically 
ensures all tests pass locally, enhancing code quality 
and preventing the accidental push of failing code.

Let's enable the pre-push hook.
We store milestones by number.

This works when filters are confined to one repo, but
doesn't make sense when saving filters across repos.

Let's refactor the milestones to save by title instead
of number
@gycgabriel gycgabriel merged commit 263426a into deploy Mar 14, 2024
3 checks passed
gycgabriel added a commit that referenced this pull request Mar 14, 2024
NereusWB922 added a commit to NereusWB922/WATcher that referenced this pull request Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants