Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
RFC 90: Sustainability roadmap #90
base: main
Are you sure you want to change the base?
RFC 90: Sustainability roadmap #90
Changes from 7 commits
744ed6d
a10fd5b
083b228
030fbec
1092bc8
b39ac8e
29287bf
adf2f0c
06cf44f
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Would it make sense to sort these row by one of the columns (at a glance, I assumed it was sorted by Reach, but the last row is 10%)?
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.
Good point. I’d probably go for "Impact" and then "Reach" if we did that?
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.
Would we consider our CSP roadmap item here. I know it's security related but from my understanding it helps browsers know when to cache things better and stops additional unexpected scripts from running in the browser. When combined with
nonce
usage, we get a nice set of improvements for both security and performance in the browser.E.g. https://techdocs.akamai.com/edgeworkers/docs/content-security-policy
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.
Good point, I don’t think I’ve seen anyone make that case but there’s probably some performance benefits
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.
Same question here, would it make sense to sort by one of the columns?
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.
Yup
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.
Would we also consider the cloud impact of Wagtail itself (our main website, docs hosting) and the CI runners?
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.
Definitely! I’ll try to think of where to add this. I’m not sure this type of consideration needs an RFC as much, but it’s good to consider.
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.
And again... although this might be sorted by complexity?
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.
Would we also consider the CSS and JS assets loading in the admin?
These may not impact the front facing parts of the website but will impact the energy usage of those accessing the admin and the servers that host the Wagtail instances.
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.
Yes we should! I’ll add it. I’ve been trying to quantify how much usage the admin gets based on internal data about Torchbox websites. It’s really hard.
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.
Should we consider our community platforms?
Maybe it's a bit too much scope (or not even measurable), but not only do we extensively use these tools but we also create an example for other communities. This might not need to be a comparison for adopting different platforms but it could involve insuring we see if the platforms actually have their own sustainability considerations/policy.
Off the top of my head; Slack, StackOverflow, Zoom (What's new in Wagtail, meetings), GitHub.
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.
Yep! Definitely. Will try to think of where to add this.
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.
I feel like there are implications from calculating Wagtail's carbon footprint on this one that might be worth noting for discussion. If we're including that in the Wagtail project's overall carbon footprint, I think there's some imprecision in any claims to net zero and risk that decisions made wholly independent of the Wagtail project could impact a net zero target.
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.
Could you expand @willbarton, on the imprecisions in particular? From my understanding you’re right, but this is the case with all carbon footprint accounting / Net Zero claims and wouldn’t be specific to Wagtail or open source more widely.
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.
Would this be controversial? Are there reasons not to follow the WSG (and possibly influence their development by attempting to put them into practice)?
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.
I see three reasons: