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 Animal Type and Add Pet Table #16

Closed
wants to merge 14 commits into from

Conversation

trinity-y
Copy link
Contributor

Fixed PascalCase for linting, added animal type and pet table.

@@ -0,0 +1,24 @@
import { DataType } from "sequelize-typescript";
Copy link
Contributor

Choose a reason for hiding this comment

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

isn't this daniel's migration file? try rebasing, this file shouldn't be in your pr

import { DataType } from "sequelize-typescript";
import { Migration } from "../umzug";

export const up: Migration = async ({ context: sequelize }) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

same here, this is jonathans

Copy link
Contributor

@aanxniee aanxniee left a comment

Choose a reason for hiding this comment

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

wait i'm confused, currently this pr is trying to merge main INTO your branch. i see that all the animal type and pet table changes are in the other pr. if you're trying to pull in the latest changes into your branch, what you want to do is

git checkout main
git pull
git checkout <your_branch>
git rebase main
git push --force-with-lease

@trinity-y trinity-y closed this Jun 29, 2024
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.

6 participants