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

Blog: The Triage Rhyme #2236

Closed
wants to merge 21 commits into from
Closed

Conversation

adamfarley
Copy link
Contributor

A fun little ditty to aid triagers in remembering one developer's take on some key aspects of the triage process.

Description of change

Checklist

  • npm test passes
  • documentation is changed or added (if applicable)
  • permission has been obtained to add new logo (if applicable)
  • contribution guidelines followed here

A fun little ditty to aid triagers in remembering one developer's
take on some key aspects of the triage process.

Signed-off-by: Adam Farley <[email protected]>
@netlify
Copy link

netlify bot commented Sep 18, 2023

Deploy Preview for eclipsefdn-adoptium ready!

Name Link
🔨 Latest commit 29bf61e
🔍 Latest deploy log https://app.netlify.com/sites/eclipsefdn-adoptium/deploys/6509aff12f0b150009ab777c
😎 Deploy Preview https://deploy-preview-2236--eclipsefdn-adoptium.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #2236 (29bf61e) into main (d659fc3) will not change coverage.
Report is 7 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2236   +/-   ##
=======================================
  Coverage   99.20%   99.20%           
=======================================
  Files          87       87           
  Lines        6515     6515           
  Branches      553      553           
=======================================
  Hits         6463     6463           
  Misses         52       52           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@adamfarley adamfarley changed the title WIP: Blog: The Triage Rhyme Blog: The Triage Rhyme Sep 19, 2023
@xavierfacq
Copy link
Member

Is it normal that all links point the file index.md? 🤔

@gdams
Copy link
Member

gdams commented Sep 19, 2023

Is it normal that all links point the file index.md? 🤔

I'm going to say no - @adamfarley what's the idea of the index.md links?

@adamfarley
Copy link
Contributor Author

Is it normal that all links point the file index.md? 🤔

I'm going to say no - @adamfarley what's the idea of the index.md links?

It's the method I used to create easily-identifiable tooltips for supplimentary information. The links themselves are incidental, so I used index.md because it keeps you on the blog page while still being a valid link (to keep the linter happy).

Happy to use a different method for the supplimentary info if anyone has ideas.

@xavierfacq
Copy link
Member

👋 Maybe you can:

  • create internal links on images or texts as anchors, links look like #something
  • use this anchors as links for your "tooltips".

e.g.

Define an anchor:

[my text or image](#tress)

Use this anchor:

is for TReSS, so go take a [gander](#tress "Identify failed pipelines via TRSS.").

@adamfarley
Copy link
Contributor Author

FYI: On vacation. I will make any changes when I return in 2 weeks, if nobody beats me to it. :)

@gdams
Copy link
Member

gdams commented Oct 24, 2023

@adamfarley merge conflicts. Do you still want this to be merged? I might suggest that it makes more sense to go in a wiki somewhere for maintainers as it has little meaning to our external users

@gdams
Copy link
Member

gdams commented Nov 4, 2023

ping @adamfarley

@gdams
Copy link
Member

gdams commented Nov 6, 2023

I'm going to close this for now, feel free to re-open it

@gdams gdams closed this Nov 6, 2023
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.

3 participants