Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: create sequelize 5/6 defs#3981

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
jedwards1211 wants to merge5 commits intoflow-typed:master
base:master
Choose a base branch
Loading
fromjedwards1211:sequelize6

Conversation

@jedwards1211
Copy link
Contributor

@AndrewSouthpaw
Copy link
Contributor

Is there a way to tell run_tests what versions of flow to use?

Not to my knowledge, short of just hacking the CLI code.

@AndrewSouthpaw
Copy link
Contributor

@jedwards1211 since this is mostly a copy-pasta job, I'm fine with merging if you've poked it enough to be satisfied.

@jedwards1211
Copy link
ContributorAuthor

Actually let's wait. I'm changing a bunch of stuff over to exact objects and spreads.

@AndrewSouthpaw
Copy link
Contributor

Sounds good! Let me know when you're ready.

@Brianzchen
Copy link
Member

@jedwards1211 any news on this?

@dannyjlaurence-exp
Copy link

We too are in need of these - anything we can do to help@jedwards1211? Even a rough outline of the exact objects or spread changes you were thinking about would point us in the right direction.

@Brianzchen
Copy link
Member

Brianzchen commentedFeb 6, 2023
edited
Loading

@dannyjlaurence-exp As far as I can tell these are new definitions which doesn't break anything existing. There's also tests in place to an extent. If you can give us some peace of mind that these defs are useful to you, I think we can merge and improve later.

Considering also this PR is 2 years old it may be very outdated against current versions of flow

@gantoinegantoine added the changes requestedChanges have been requested by a reviewer labelMay 11, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

changes requestedChanges have been requested by a reviewer

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jedwards1211@AndrewSouthpaw@Brianzchen@dannyjlaurence-exp@gantoine

[8]ページ先頭

©2009-2025 Movatter.jp