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

[4/7] refactor(core): re-export formatting symbols from cursive_core #1356

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

arxanas
Copy link
Owner

@arxanas arxanas commented Jun 30, 2024

Stack:


refactor(core): re-export formatting symbols from cursive_core

This should have been done from the beginning, since we operate on these symbols in several places for formatting. This would also help a migration away from cursive, since now only two modules have direct dependencies on cursive/cursive_core:

  • git-branchless-lib for general output formatting.
  • git-branchless-undo for the interactive undo UI.

@arxanas arxanas changed the title refactor(core): re-export formatting symbols from cursive_core [4/6] refactor(core): re-export formatting symbols from cursive_core Jun 30, 2024
@arxanas arxanas changed the base branch from master to arxanas/feat-submit-phabricator-pass-allow-untracked-to-arc June 30, 2024 20:39
@arxanas arxanas changed the title [4/6] refactor(core): re-export formatting symbols from cursive_core [4/7] refactor(core): re-export formatting symbols from cursive_core Jun 30, 2024
Base automatically changed from arxanas/feat-submit-phabricator-pass-allow-untracked-to-arc to master July 2, 2024 14:47
@arxanas arxanas force-pushed the arxanas/refactor-core-re-export-formatting-symbols-from-cursive-core branch 2 times, most recently from 0cf0817 to 94889b3 Compare July 6, 2024 18:38
It seems easier for contributors to get started if the compiler version is already configured; plus, this will presumably make it so that I don't have to set the Rust version with `rustup override` whenever I create a new worktree.
This should have been done from the beginning, since we operate on these symbols in several places for formatting. This would also help a migration away from `cursive`, since now only two modules have direct dependencies on `cursive`/`cursive_core`:

- git-branchless-lib for general output formatting.
- git-branchless-undo for the interactive undo UI.
@arxanas arxanas force-pushed the arxanas/refactor-core-re-export-formatting-symbols-from-cursive-core branch from 94889b3 to 0d83fd1 Compare August 9, 2024 01:37
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