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

Revert "feat: fix package structure with did modules" #816

Merged
merged 3 commits into from
Oct 31, 2023
Merged

Conversation

prnk28
Copy link
Contributor

@prnk28 prnk28 commented Oct 31, 2023

This reverts commit 102f276.

feat: fix package structure with did modules

Remove/pkg cli deps (#815)

  • feat: remove cli pkgs
  • Revert "feat: remove cli pkgs"

This reverts commit 858057e.

  • feat: swagger docs
  • feat: update lints

Signed-off-by: Prad Nukala prad@sonr.io

fix: goreleaser status

Revert "Remove/pkg cli deps (#815)"

This reverts commit 43c28f8.

This reverts commit 102f276.

feat: fix package structure with did modules

Remove/pkg cli deps (#815)

* feat: remove cli pkgs

* Revert "feat: remove cli pkgs"

This reverts commit 858057e.

* feat: swagger docs

* feat: update lints

---------

Signed-off-by: Prad Nukala <prad@sonr.io>

fix: goreleaser status

Revert "Remove/pkg cli deps (#815)"

This reverts commit 43c28f8.
Copy link

@senior-dev-bot senior-dev-bot bot left a comment

Choose a reason for hiding this comment

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

Feedback from Senior Dev Bot

.goreleaser.yaml Outdated
Comment on lines 172 to 178

name_template: "Sonr - v{{.Version}} 🆔"
mode: replace
draft: true
draft: false
extra_files:
- glob: ./scripts/install.sh

Choose a reason for hiding this comment

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

The code changes look good. However, I do have a suggestion to improve the readability of the name_template value. Instead of using a string concatenation, you can use the fmt package to format the string with placeholders and pass the Version value when needed. This will make it easier to understand the logic and make any future modifications.

name_template: fmt.Sprintf("Sonr - v%d 🆔", .Version)

Signed-off-by: Prad Nukala <prad@sonr.io>
@what-the-diff
Copy link

what-the-diff bot commented Oct 31, 2023

PR Summary

  • Release Settings Update
    The setting related to draft releases in the configuration file has been switched from true to false. This change means future software releases will no longer default to a draft status; instead, they will be published immediately after being created. This can lead to more efficient and streamlined release processes.

This reverts commit 334d2fd, reversing
changes made to 0bb2625.
@sweep-ai
Copy link

sweep-ai bot commented Oct 31, 2023

Apply Sweep Rules to your PR?

  • Apply: Leftover TODOs in the code should be handled.
  • Apply: All new business logic should have corresponding unit tests in the tests/ directory.
  • Apply: Any clearly inefficient or repeated code should be optimized or refactored.

@prnk28 prnk28 merged commit 254d29e into master Oct 31, 2023
6 of 12 checks passed
@prnk28 prnk28 deleted the revert/v0.8.0 branch October 31, 2023 07:27
@sonarcloud
Copy link

sonarcloud bot commented Oct 31, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 3 Security Hotspots
Code Smell A 45 Code Smells

0.0% 0.0% Coverage
5.3% 5.3% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@prnk28 prnk28 added this to the Public Testnet Deployment milestone Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant