Skip to content

feat: Make integration #4083

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

Open
wants to merge 483 commits into
base: master
Choose a base branch
from
Open

Conversation

vporton
Copy link
Contributor

@vporton vporton commented Jan 28, 2025

Description

  • Added flags like --no-compile to deploy and generate and --no-deps (don't compile dependencies) to build.
  • Added command rules that outputs GNU Make rules.

Then rules can be used:

  • build@NAME - compile a canister.
  • deploy@NAME - deploy a canister.
  • generate@NAME - generate bindings for a canister.

Implementation note: I added argument no_deps to many functions. This is a non-elegant code and I am asking you to advise me what can be done.

How Has This Been Tested?

I ran it literally at least hundreds (or thousands) of times in practical uses of my software. It does work well.

Checklist:

  • The title of this PR complies with Conventional Commits.
  • I have edited the CHANGELOG accordingly.
  • I have made corresponding changes to the documentation.

@vporton
Copy link
Contributor Author

vporton commented Jan 30, 2025

After fixing some bugs, I successfully deployed my site in production using my modified DFX.

@vporton
Copy link
Contributor Author

vporton commented Apr 29, 2025

Please, review this PR for its merge, because I tested its last version hundreds of time in a real scenario. It is ready to be merged.

@vporton vporton mentioned this pull request Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant