-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Document Examples of using GitHub Alerts (Obsidian Callouts) #750
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
base: main
Are you sure you want to change the base?
Conversation
vburzynski
commented
Apr 18, 2025
•
edited
Loading
edited
- Adds a template example of using the GitHub Alert or Obsidian Callout Syntax
- Documents the discrepancies between the two implementations
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR descriptions explains the what, but I'd love to see a bit more about the why. Can you explain why you think this is needed as part of the guides repo? Are you trying to create some kind of style guide for this repo? For all thoughtbot work on GitHub?
If this is more of a how-to article, maybe it's a better fit for something like the blog?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should not be committed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doh, I always expect those to be git ignore. I'll add that
@JoelQ Oh, good point. I didn't fully share all my thinking. I'm now realizing I didn't have a fully formed one yet. My process on Friday largely went like this:
The main thing is that I think the Alert/Callout syntax would:
|