Skip to content

added flaskit up command#2

Merged
MaswiliK merged 2 commits intoMaswiliK:mainfrom
mainulsunvi:fix/local-dev-automation
Feb 18, 2026
Merged

added flaskit up command#2
MaswiliK merged 2 commits intoMaswiliK:mainfrom
mainulsunvi:fix/local-dev-automation

Conversation

@mainulsunvi
Copy link
Contributor

Hi @MaswiliK

I've successfully resolved Issue #1

Please take a moment to review it on your end. If everything looks good, I'd appreciate a quick merge. Thank you!

@MaswiliK
Copy link
Owner

@mainulsunvi
Summary
The up command implementation is mostly correct. The main things to address before merging:

  • Remove dead run_step function
  • Replace the new command copy-paste with app.command("new")(create)
  • Fix the trailing space in the "Next steps" output
  • Split formatting into its own commit
  • Consider adding init.py to generated blueprint directories so up's DB init works

@mainulsunvi
Copy link
Contributor Author

Hello @MaswiliK

I've made updates to the code; please take a look.

@MaswiliK
Copy link
Owner

@mainulsunvi bare with me, I have been doing updates to the repo and ci workflows for a standard code structure. Have a look at the errors ( Platform independence is the goal ).

@mainulsunvi
Copy link
Contributor Author

Hello @MaswiliK,

Could you please confirm whether Issue #1 has been resolved?

I’ve noticed several errors in the checks that are not caused by my code. While I can address those issues, it would be more efficient if you could grant me Collaborator access to this repository.

Additionally, if the flaskit up command is ready for deployment, I kindly request that you merge it as soon as possible.

Thank you!

@MaswiliK MaswiliK merged commit e16344c into MaswiliK:main Feb 18, 2026
10 of 14 checks passed
@MaswiliK
Copy link
Owner

Cheers @mainulsunvi

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.

2 participants