-
Notifications
You must be signed in to change notification settings - Fork 11
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
Added yml config for Green Metrics Tool #587
Added yml config for Green Metrics Tool #587
Conversation
associated test.
Thanks for this @jcamilleri-scottlogic - as discussed, I'll add the necessary changes to run the django container in the setup |
ok, once #588 is merged in I think we'll be in shape to update the usage scenarios for testing in GMT: Some feedback from Arne at Green Coding Solutions:
@jcamilleri-scottlogic - I'll make the commits to this branch as I work to get the usage scenarios generating useable numbers. |
@jcamilleri-scottlogic - I don't think I can make commits to this branch, so I've made a new PR, pulling in your commit, so that you folks are recognised for working on the PR: I've also added you as a collaborator for the time being, so you are able to make pushes to that PR/branch if need be. |
Thanks Chris. I saw Arne's comments about the YML file - I'll see if I can get those parts fixed up this afternoon.
From: Chris Adams ***@***.***>
Sent: Wednesday, May 15, 2024 11:53 AM
To: thegreenwebfoundation/admin-portal ***@***.***>
Cc: James Camilleri ***@***.***>; Mention ***@***.***>
Subject: Re: [thegreenwebfoundation/admin-portal] Added yml config for Green Metrics Tool (PR #587)
@jcamilleri-scottlogic<https://github.com/jcamilleri-scottlogic> - I don't think I can make commits to this branch, so I've made a new PR, pulling in your commit, so that you folks are recognised for working on the PR:
#589<#589>
I've also added you as a collaborator for the time being, so you are able to make pushes to that PR/branch if need be.
-
Reply to this email directly, view it on GitHub<#587 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AYEC3RLNND7SLHAGUJKPZATZCM5B5AVCNFSM6AAAAABHQOJ7T6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGE4DMMRVGA>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
The contents of this email and any attachments are intended solely for the addressee and may contain confidential or legally privileged information. If you have received this message in error, please send it back to us, and immediately and permanently delete it. The information may not be used or disclosed except for the purpose for which it has been sent.
Email is susceptible to data corruption, interception, unauthorised amendment, viruses, and unforeseen delays. Although Scott Logic Limited has taken reasonable precautions to avoid these situations, it cannot accept responsibility for any loss or damage sustained as a result of any of these actions and the recipient must ensure that the email (and attachments) are virus-free.
Please note, that we do not accept notification of changes to bank account details by email. This applies to notifications from or to us.
Scott Logic Limited is a limited company registered in England and Wales with registration number 05377430. Registered office address: 6th Floor, The Lumen, St James Boulevard, Newcastle Helix, Newcastle upon Tyne, NE4 5BZ . Our VAT number is 866 1051 30.
|
@jcamilleri-scottlogic - I'm making notes as I work in #589 - it might be worth comparing notes there |
I'm closing this in favour of the other PR #589 where @jcamilleri-scottlogic continued to work on this. |
associated test.