Skip to content

fix(docs): correct SDK package links and add template cleanup note #602

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
Merged
roderik merged 1 commit into from
Jul 18, 2025
Merged

fix(docs): correct SDK package links and add template cleanup note #602

roderik merged 1 commit into from
Jul 18, 2025

Conversation

@SaeeDawod
Copy link
Contributor

@SaeeDawod SaeeDawod commented Jul 18, 2025 •
edited by sourcery-ai bot
Loading

  • Fix SDK package links to point to GitHub repo folders instead of internal docs
  • Add note about removing unused sample contracts from Empty template
  • Addresses customer feedback about Counter contract references

Summary by Sourcery

Fix SDK documentation by updating package links to point to the GitHub repository and add a note on removing unused sample contracts from the Empty template

Documentation:

  • Updated SDK package links to point to GitHub repository folders instead of internal docs
  • Added an informational callout advising removal of unused sample contracts (e.g., Counter) from the Empty template before production

@saeeddawod
fix(docs): correct SDK package links and add template cleanup note
d3841b0 
- Fix SDK package links to point to GitHub repo folders instead of internal docs
- Add note about removing unused sample contracts from Empty template
- Addresses customer feedback about Counter contract references
@github-actions github-actions bot added the qa:running QA workflow is currently running label Jul 18, 2025
sourcery-ai[bot]
sourcery-ai bot reviewed Jul 18, 2025
View reviewed changes
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We've reviewed this pull request using the Sourcery rules engine

@github-actions
Copy link

github-actions bot commented Jul 18, 2025 •
edited
Loading

To view in Slack, search for: 1752818173.184999

@github-actions github-actions bot added status:ready-for-review Pull request is ready for review qa:success QA workflow passed successfully fix Bug fix and removed qa:running QA workflow is currently running labels Jul 18, 2025
@roderik roderik merged commit a6a5ab9 into main Jul 18, 2025
10 checks passed
@roderik roderik deleted the fix/sdk-links-and-template-notes branch July 18, 2025 10:13
@github-actions github-actions bot added status:merged Pull request has been merged and removed status:ready-for-review Pull request is ready for review labels Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sourcery-ai sourcery-ai[bot] sourcery-ai[bot] left review comments

@bl0up bl0up Awaiting requested review from bl0up

@robbeverhelst robbeverhelst Awaiting requested review from robbeverhelst

@gyan-sharma gyan-sharma Awaiting requested review from gyan-sharma

Assignees

No one assigned

Labels

fix Bug fix qa:success QA workflow passed successfully status:merged Pull request has been merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@SaeeDawod @roderik @saeeddawod