Skip to content

Conversation

@WilliamBergamin
Copy link
Contributor

Type of change

  • New sample
  • New feature
  • Bug fix
  • Documentation

Summary

This PR aims to update the sample to the Deno 2 standards and the latest CLI standards.

  • Move content of slack.json to .slack/hooks.json
  • Remove dependency on mock-fetch/
  • Import from JSR as much as possible
  • Move imports from import-map.json to deno.jsonc

Requirements

  • I’ve checked my submission against the Samples Checklist to ensure it complies with all standards
  • I have ensured the changes I am contributing align with existing patterns and have tested and linted my code
  • I've read and agree to the Code of Conduct

@WilliamBergamin WilliamBergamin requested a review from zimeg May 23, 2025 19:03
@WilliamBergamin WilliamBergamin self-assigned this May 23, 2025
@WilliamBergamin WilliamBergamin added the update PR only - an update or maintenance is being made to an existing app label May 23, 2025
Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

Starting with the latest is a great feeling! 🚀 ✨

Comment on lines +52 to +53
const updatedMsg =
`:wave: <@${inputs.user}> submitted the following message: \n\n>${inputs.message}`;
Copy link
Member

Choose a reason for hiding this comment

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

@WilliamBergamin WilliamBergamin merged commit f3383ae into main May 29, 2025
2 checks passed
@WilliamBergamin WilliamBergamin deleted the update-with-new-standards branch May 29, 2025 14:46
@WilliamBergamin WilliamBergamin mentioned this pull request May 29, 2025
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

update PR only - an update or maintenance is being made to an existing app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants