-
Notifications
You must be signed in to change notification settings - Fork 105
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
add requested changes from KubeCon tag project meeting #83
Conversation
mkorbi
commented
Apr 23, 2023
- Add GitHub Repo to page
- Add Glossary to page
- Add Glossary link to landscape
✅ Deploy Preview for tag-env-sustainability ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Added also a glossary. Not sure about the right-side menu with the letters. |
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.
Nice, I like that we move the glossary to the website.
documentation/WhitePapers/SustainabilityUseCasesAndLandscape2023.md
Outdated
Show resolved
Hide resolved
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.
Looking pretty good. Some minor requests for wording changes for clarity.
documentation/WhitePapers/SustainabilityUseCasesAndLandscape2023.md
Outdated
Show resolved
Hide resolved
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.
LGTM, we need to rebase it tho
requested changes have been addressed
@mkorbi could you take a look |
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.
One not. Address then I'll approve!
Scope 3: indirect emissions, that not directly can be changed: for example delivery chain caused CO2 pollution | ||
|
||
|
||
#### **Embodied (Carbon) Emissions** |
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.
Swap this with the emissions. Wrong order.
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.
LGTM