Repository navigation
Add legal notices to the reports and mark the edition - #105
Closed
bencehezso wants to merge 82 commits into
Closed
bencehezso wants to merge 82 commits into
bencehezso wants to merge 82 commits into
Conversation
Add AGENTS.md with contributor and agent working rules
Foundation for CLI 2.0: exit codes, config.py hygiene, dependency refresh
Add configuration precedence, key files and the config command
Update the catalogue through its manifest and add catalogue commands
Build the sync document from the API v2 definition
Always compute and write the local report, and only record upload links
Replace the PDF with the Basic Executive Summary
Add the API v2 client and remove the v1 client
Ship the report assets inside the installed package
Count Azure resources without a location as global
Identify release/2.0 builds as 2.0.0.dev0
Make Azure imports lazy and fix upgrades from 1.x environments
Remove the shared icon images and draw a placeholder instead
Remove unused assets and legacy report code
Break the egress cycle with a model module and move tfstate into the package
Fold the report modules into the cloudexit.report package
Move the platform client, sync document, upload and spool into the cloudexit package
Move the stage engine into the cloudexit package
Move the command line into cloudexit.cli and install a single package
Move the terminal helpers to a leaf module and guard the package layering
Name every logger after its module
Split the argument parser out of cloudexit.cli.app
Split the catalogue download code out of the store
…tials Stop writing credential identifiers into the egress PDF and run.log
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Every Executive Summary (PDF and HTML) now ends with a Legal Notices section: the Community Edition disclaimer, what the report is and is not, the icon notice for the report's own provider, trademark notices, the AGPL-3.0 line with the repository link, and the copyright line. The cover and the running footer show the edition instead of "Confidential". The notices are built in one shared module, so the egress report can use them too.
Changes
Wording
Notices
cloudexit/report/notices.pybuilds the notices once as plain data; the PDF and the HTML templates render that same data. Only the icon notice of the report's provider is printed.PDF
HTML
index.htmlandegress.htmlend with the same notices section; the egress page takes the year from the estimate's timestamp.README
LICENSEis unchanged.Tests
Compatibility
Config, exit codes, the dataset and the API v2 definition copy are unchanged. Report output: a new last page and HTML section, the cover and footer show the edition instead of "Confidential", and the egress PDF is unchanged until the next change. The PDF metadata is unchanged.