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

Filesystem caching #277

Closed
wants to merge 4 commits into from
Closed

Filesystem caching #277

wants to merge 4 commits into from

Conversation

matthewmueller
Copy link
Contributor

@matthewmueller matthewmueller commented Sep 5, 2022

One of the great things about building a framework and having control over the build system is that we can implement very good caching. So far the caching has been limited to not re-running generators within a single sync, but when you call bud run there's a sync every time you change a Go file.

Syncing is still quite fast, but I'd like to be able to spawn subcommands during sync to enable features like custom generators. Restarting processes each sync is time-consuming. Instead we'll only restart processes when dependencies of the custom generators change (almost never).

Caching is quite hard to get right, so we may run into some stale builds along the way, but I'm confident these issues can be ironed out over time.

Next steps:

  • Each change should be tested against directed acyclic graph (dag) of globs
    This will allow the controller generator to react to a new file in view/**
  • Get the controller tests passing
  • Manual QA

@matthewmueller matthewmueller marked this pull request as draft September 5, 2022 06:17
@matthewmueller
Copy link
Contributor Author

Closing in favor of: #281

@matthewmueller matthewmueller deleted the incremental-caching branch September 14, 2022 20:10
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.

1 participant