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.

Sign up for GitHub

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

Jump to bottom

fix: prevent unaffected rust files from getting formatted #281

Merged
merged 4 commits into from
May 13, 2024
Merged

fix: prevent unaffected rust files from getting formatted #281

merged 4 commits into from
May 13, 2024

Conversation

c12i
Copy link
Collaborator

@c12i c12i commented May 13, 2024

This PR fixes #261

With regards to the scaffolding tool removing comments, I think it's easier if this is tackled in a separate PR. I have re-opened the original issue to keep track of it as well #167 since the root cause is now known.

@c12i c12i requested a review from matthme May 13, 2024 00:54
matthme
matthme approved these changes May 13, 2024
View reviewed changes
Copy link
Collaborator

@matthme matthme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@c12i c12i added ShouldBackport/0.3 This change should be backported to develop-0.3 ShouldBackport/0.2 This change should be backported to develop-0.2 labels May 13, 2024
@c12i c12i merged commit c36532f into holochain:develop May 13, 2024
3 of 7 checks passed
c12i added a commit that referenced this pull request May 16, 2024
@c12i
chore: bump weekly (#286)
941dc28 
* fix: invalid entry type fields in vue template (#279)

* fix: prevent unaffected rust files from getting formatted (#281)

* fix: preserve doc comments

* fix: unmodified files getting reformatted by unparse

* refactor map_all_files_rec function

* revert unparse function change

* fix: prevent scaffolding dead code (#280)

* fix: prevent scaffolding dead code

* fix clippy warnings

* improve readability of stringified rust code

* update coordinator functions

* refactor boolean checks

* feat: Add headless template (#267)

* feat: add initial headless template

* include headless template in ui framework choices

* add web-app instructions and prefer explicit hc commands in all package.json templates

* rename instructions template

* fix indentation in instructions

* update instructions

* remove gitkeep

* refactor UiFramework struct

* refactor tempalte config access

* remove assertion

* revert renaming

* revert integrity zome name

* choose non vanilla framework for non hello world examples

* simplify ui framework prompts; update instructions

* update instructions

* Update scripts and instructions

* color code ui frameworks

* templatify web-happ manifest

* fix bug writing invalid template config

* remove unnecessary print statement

* remove unnecessary return statement

* update ui framework try from filetree implementation

* refactor: refactor reserved words check

* update ansi colors for lit and svelte

* Add build happ instruction

* chore: update web-app instructions for headless template

* fix: fix vue example ui server not starting (#284)

* simplify templates_dir

* chore: bump weekly
@c12i c12i removed the ShouldBackport/0.3 This change should be backported to develop-0.3 label May 16, 2024
c12i added a commit to c12i/scaffolding that referenced this pull request May 16, 2024
@c12i
fix: prevent unaffected rust files from getting formatted (holochain#281
d775007 
)

* fix: preserve doc comments

* fix: unmodified files getting reformatted by unparse

* refactor map_all_files_rec function

* revert unparse function change
c12i added a commit that referenced this pull request May 16, 2024
@c12i
chore: bump and release 0.3 (#287)
5c5e19c 
* feat: Add headless template (#267)

* feat: add initial headless template

* include headless template in ui framework choices

* add web-app instructions and prefer explicit hc commands in all package.json templates

* rename instructions template

* fix indentation in instructions

* update instructions

* remove gitkeep

* refactor UiFramework struct

* refactor tempalte config access

* remove assertion

* revert renaming

* revert integrity zome name

* choose non vanilla framework for non hello world examples

* simplify ui framework prompts; update instructions

* update instructions

* Update scripts and instructions

* color code ui frameworks

* templatify web-happ manifest

* fix bug writing invalid template config

* remove unnecessary print statement

* remove unnecessary return statement

* update ui framework try from filetree implementation

* refactor: refactor reserved words check

* update ansi colors for lit and svelte

* Add build happ instruction

* fix: invalid entry type fields in vue template (#279)

* fix: prevent scaffolding dead code (#280)

* fix: prevent scaffolding dead code

* fix clippy warnings

* improve readability of stringified rust code

* update coordinator functions

* refactor boolean checks

* fix: prevent unaffected rust files from getting formatted (#281)

* fix: preserve doc comments

* fix: unmodified files getting reformatted by unparse

* refactor map_all_files_rec function

* revert unparse function change

* fix: fix vue example ui server not starting (#284)

* bump version
c12i added a commit that referenced this pull request May 20, 2024
@c12i
chore: Bump weekly version (#291)
4b2116e 
* fix: invalid entry type fields in vue template (#279)

* fix: prevent unaffected rust files from getting formatted (#281)

* fix: preserve doc comments

* fix: unmodified files getting reformatted by unparse

* refactor map_all_files_rec function

* revert unparse function change

* fix: prevent scaffolding dead code (#280)

* fix: prevent scaffolding dead code

* fix clippy warnings

* improve readability of stringified rust code

* update coordinator functions

* refactor boolean checks

* feat: Add headless template (#267)

* feat: add initial headless template

* include headless template in ui framework choices

* add web-app instructions and prefer explicit hc commands in all package.json templates

* rename instructions template

* fix indentation in instructions

* update instructions

* remove gitkeep

* refactor UiFramework struct

* refactor tempalte config access

* remove assertion

* revert renaming

* revert integrity zome name

* choose non vanilla framework for non hello world examples

* simplify ui framework prompts; update instructions

* update instructions

* Update scripts and instructions

* color code ui frameworks

* templatify web-happ manifest

* fix bug writing invalid template config

* remove unnecessary print statement

* remove unnecessary return statement

* update ui framework try from filetree implementation

* refactor: refactor reserved words check

* update ansi colors for lit and svelte

* Add build happ instruction

* chore: update web-app instructions for headless template

* fix: fix vue example ui server not starting (#284)

* simplify templates_dir

* chore: update manual testing docs (#285)

* chore: update manual testing docs

* update override

* fix: bug while scaffolding collection by author (#290)

* fix: bug while scaffolding collection by author

* fix initial insert statement

* chore: bump weekly
@c12i c12i removed the ShouldBackport/0.2 This change should be backported to develop-0.2 label Jun 5, 2024
c12i pushed a commit to c12i/scaffolding that referenced this pull request Jun 5, 2024
@matthme @c12i
fix: prevent unaffected rust files from getting formatted (holochain#281
98db25f 
)

* fix: preserve doc comments

* fix: unmodified files getting reformatted by unparse

* refactor map_all_files_rec function

* revert unparse function change
c12i added a commit that referenced this pull request Jun 6, 2024
@c12i @matthme
chore: backport and bump scaffolding 0.2000.7 (#305)
6f3a5b6 
* fix: Ensure is holo env var is passed at build time (#251)

* fix: Ensure is holo env var is passed at build time

* fix: only pass the env var is holo is enabled

* have separate package scripts

* Remove root package holo script

* fix: fix json parse error

* chore: Update templates (#259)

* fix: remove instances of file_exists helper from custom template

* fix: remove redundant package:holo script from vue template

* fix: invalid entry type fields in vue template (#279)

* chore: bump version

* fix: prevent unaffected rust files from getting formatted (#281)

* fix: preserve doc comments

* fix: unmodified files getting reformatted by unparse

* refactor map_all_files_rec function

* revert unparse function change

---------

Co-authored-by: matthme <36768177+matthme@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthme matthme matthme approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Disable reformatting of unaffected files
2 participants
@c12i @matthme