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

fix: Fix misplaced doc comments #411

Merged
merged 2 commits into from
Nov 8, 2024
Merged

Conversation

c12i
Copy link
Collaborator

@c12i c12i commented Nov 5, 2024

Fixes #406
Fixes #413 (ensures // in URLs are not inadvertently replaced)

This PR also contains some additional code refactors to how the validate function codegen is handled

@c12i c12i marked this pull request as ready for review November 7, 2024 14:39
@c12i c12i requested a review from matthme November 7, 2024 14:39
@c12i c12i force-pushed the fix/clippy-warnings-code-comments branch from c28f304 to ddc2026 Compare November 8, 2024 00:04
@c12i c12i requested a review from matthme November 8, 2024 00:06
@c12i
Copy link
Collaborator Author

c12i commented Nov 8, 2024

CI failures to be addressed when a newer version of tryorama is released

@c12i c12i merged commit 6d984d2 into develop Nov 8, 2024
2 of 8 checks passed
@c12i c12i deleted the fix/clippy-warnings-code-comments branch November 8, 2024 11:34
@c12i c12i added the ShouldBackport/0.4 This change should be backported to develop-0.1 label Nov 8, 2024
@c12i c12i added ShouldBackport/0.3 This change should be backported to develop-0.3 and removed ShouldBackport/0.3 This change should be backported to develop-0.3 labels Nov 19, 2024
c12i added a commit that referenced this pull request Nov 19, 2024
* Fix misplaced doc comments

* Slightly refactor link_type integrity codegen
@c12i c12i removed the ShouldBackport/0.4 This change should be backported to develop-0.1 label Nov 26, 2024
c12i added a commit that referenced this pull request Nov 26, 2024
* Fix misplaced doc comments

* Slightly refactor link_type integrity codegen
c12i added a commit that referenced this pull request Nov 26, 2024
* Bump versions

* Nix flake update

* fix: Refactor/custom templates (#397)

* Fix custom template flake

* Refactor custom templates

* chore: Update holochain dependencies (#398)

* Update dependencies

* Remove unused dependency

* fix: Skip writing scaffold config for nixified custom templates (#415)

* Add skip_config_check global flag to scaffolding

* Prefer not writing the scaffold config for nixified custom templates

* Fix rustfmt warning

* fix: Fix misplaced doc comments (#411)

* Fix misplaced doc comments

* Slightly refactor link_type integrity codegen

* Remove instances of SignalType

* feat: further improve ci runtime (#414)

* Remove dependency to holochain

* Add rust-cache

* Add cachix nix config

* Add ci job concurrency options

* Update cachix configurations

* Run rust checks/tests outside nix

* add missing components

* Cache rust builds

* Bump holochain deps

* Update rustdocs

* Build cargo deps separately

* refactor flake

* Update maintenance docs

* Add extraPullNames input to cachix actions

* Add nix_build step

* Extend extra-substituters and public-keys in nixConfig

* Fix ci workflow

* Bump client-js version
c12i added a commit that referenced this pull request Dec 10, 2024
* Fix misplaced doc comments

* Slightly refactor link_type integrity codegen
c12i added a commit that referenced this pull request Dec 12, 2024
* fix: Skip writing scaffold config for nixified custom templates (#415)

* Add skip_config_check global flag to scaffolding

* Prefer not writing the scaffold config for nixified custom templates

* Fix rustfmt warning

* feat: further improve ci runtime (#414)

* Remove dependency to holochain

* Add rust-cache

* Add cachix nix config

* Add ci job concurrency options

* Update cachix configurations

* Run rust checks/tests outside nix

* add missing components

* Cache rust builds

* Bump holochain deps

* Update rustdocs

* Build cargo deps separately

* refactor flake

* Update maintenance docs

* Add extraPullNames input to cachix actions

* Add nix_build step

* Extend extra-substituters and public-keys in nixConfig

* Fix ci workflow

* Bump client-js version

* fix: Fix misplaced doc comments (#411)

* Fix misplaced doc comments

* Slightly refactor link_type integrity codegen

* feat: Enhance custom template generation (#404)

* feat: Add a new template subcommand to create new custom template bases
from existing templates

* Remove call to wrapCustoTemplate in scaffolding nix flake

* Update custom template docs

* chore: update ui dependencies (#407)

* Bump holochain playground

* Update ui core dependencies

* Update lit dependencies

* Fix cli output lines

* Fix broken css import in lit template

* Revert custom-template changes

* Fix vue templates

* Increase testTimeout

* Remove eslint from lit template

* Fix lit CI failure

* Add missing shared styles

* feat: Reprompt user on invalid input (#402)

* Fix imports

* Merge pull request #428 from holochain/fix/svelte-client-race-condition-0.4

chore: backport svelte race condition to 0.4 and release 0.4000.0-rc.1

* feat: Add ability to go back and modify entry type fields (#418)

* Add go back functionality for hc scaffold entry-type

* address PR comments

* Ensure the fields vec is not empty on change

* Improve text spacing

* Simplify prompt

* Pin typescript version for vue-tsc compatibility

---------

Co-authored-by: matthme <[email protected]>
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.

[BUG] Broken link in generated validate function comments Many clippy warnings in scaffolded dnas
2 participants