[WIP] [WIP] Address feedback from review on 'Smoke' pull request#80
Merged
[WIP] [WIP] Address feedback from review on 'Smoke' pull request#80
Conversation
Closed
Co-authored-by: FMI-Test <24483979+FMI-Test@users.noreply.github.com> Agent-Logs-Url: https://github.com/FMI-Test/TomWizMaster/sessions/753da305-92ad-4422-a953-41b7bf44aa43
Contributor
There was a problem hiding this comment.
Pull request overview
Updates Fire/README.md to redact sensitive/specific organizational references from the “Smoke” content while keeping the narrative and operational examples largely intact.
Changes:
- Replaces explicit company/organization names with generalized descriptors (systems integrator, global enterprise, generic BUs).
- Generalizes defense/aerospace references in the migration/compliance narrative.
- Adjusts the “production fires” example to use generic BU names instead of named entities.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|  | ||
|
|
||
| Lead Org of 350 FTE & Contractors as a Project Lead (My Official Title in TechM) in Tech Mahindra and contracted TO GE ... Soon I owned the end-to-end CoreTech Tier 0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech, then Migration of 100s of AWS Accounts and Orgs and this SH* & TH* SH* to GE Vernova (Energy & Nuclear) with Comm & GOV-Cloud AWS Accounts etc. GE HealthCare with its CT Scans and ... to GE Aerospace which makes Jet Engines for DoD, Air Force One, & Boeing hence Gov-Cloud Regions & Regulation with NIST, Compliance, Cyber, GuardRails and coding all as fully automated a XXLarge Scale Mission Impossible. | ||
| Led an org of 350+ FTEs & Contractors as Project Lead at a major systems integrator, contracted to a global industrial conglomerate. Soon owned end-to-end Tier-0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech — then migration of 100s of AWS Accounts and Orgs across multiple business units (Energy & Nuclear, HealthCare, and Aerospace / Defense) with Commercial & Gov-Cloud AWS Accounts, NIST, Compliance, Cyber, GuardRails, and fully automated IaC at an XXLarge scale — a Mission Impossible delivered. |
There was a problem hiding this comment.
The redaction still leaves fairly specific privileged-access and regulated-cloud details (e.g., “owned end-to-end Tier-0 Access, Control, & Authority” and “Commercial & Gov-Cloud AWS Accounts”). If the intent is to remove sensitive operational/security specifics from public docs, consider generalizing further (e.g., “privileged access management” / “regulated cloud environments”) and dropping the explicit Tier-0 / Gov-Cloud phrasing here.
|  | ||
|
|
||
| Lead Org of 350 FTE & Contractors as a Project Lead (My Official Title in TechM) in Tech Mahindra and contracted TO GE ... Soon I owned the end-to-end CoreTech Tier 0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech, then Migration of 100s of AWS Accounts and Orgs and this SH* & TH* SH* to GE Vernova (Energy & Nuclear) with Comm & GOV-Cloud AWS Accounts etc. GE HealthCare with its CT Scans and ... to GE Aerospace which makes Jet Engines for DoD, Air Force One, & Boeing hence Gov-Cloud Regions & Regulation with NIST, Compliance, Cyber, GuardRails and coding all as fully automated a XXLarge Scale Mission Impossible. | ||
| Led an org of 350+ FTEs & Contractors as Project Lead at a major systems integrator, contracted to a global industrial conglomerate. Soon owned end-to-end Tier-0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech — then migration of 100s of AWS Accounts and Orgs across multiple business units (Energy & Nuclear, HealthCare, and Aerospace / Defense) with Commercial & Gov-Cloud AWS Accounts, NIST, Compliance, Cyber, GuardRails, and fully automated IaC at an XXLarge scale — a Mission Impossible delivered. |
There was a problem hiding this comment.
This line uses “HealthCare” (camel-case). For readability and consistent English usage, consider standardizing to “Healthcare” (and keep the same casing anywhere else this term appears).
Suggested change
| Led an org of 350+ FTEs & Contractors as Project Lead at a major systems integrator, contracted to a global industrial conglomerate. Soon owned end-to-end Tier-0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech — then migration of 100s of AWS Accounts and Orgs across multiple business units (Energy & Nuclear, HealthCare, and Aerospace / Defense) with Commercial & Gov-Cloud AWS Accounts, NIST, Compliance, Cyber, GuardRails, and fully automated IaC at an XXLarge scale — a Mission Impossible delivered. | |
| Led an org of 350+ FTEs & Contractors as Project Lead at a major systems integrator, contracted to a global industrial conglomerate. Soon owned end-to-end Tier-0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech — then migration of 100s of AWS Accounts and Orgs across multiple business units (Energy & Nuclear, Healthcare, and Aerospace / Defense) with Commercial & Gov-Cloud AWS Accounts, NIST, Compliance, Cyber, GuardRails, and fully automated IaC at an XXLarge scale — a Mission Impossible delivered. |
|  | ||
|
|
||
| Lead Org of 350 FTE & Contractors as a Project Lead (My Official Title in TechM) in Tech Mahindra and contracted TO GE ... Soon I owned the end-to-end CoreTech Tier 0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech, then Migration of 100s of AWS Accounts and Orgs and this SH* & TH* SH* to GE Vernova (Energy & Nuclear) with Comm & GOV-Cloud AWS Accounts etc. GE HealthCare with its CT Scans and ... to GE Aerospace which makes Jet Engines for DoD, Air Force One, & Boeing hence Gov-Cloud Regions & Regulation with NIST, Compliance, Cyber, GuardRails and coding all as fully automated a XXLarge Scale Mission Impossible. | ||
| Led an org of 350+ FTEs & Contractors as Project Lead at a major systems integrator, contracted to a global industrial conglomerate. Soon owned end-to-end Tier-0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech — then migration of 100s of AWS Accounts and Orgs across multiple business units (Energy & Nuclear, HealthCare, and Aerospace / Defense) with Commercial & Gov-Cloud AWS Accounts, NIST, Compliance, Cyber, GuardRails, and fully automated IaC at an XXLarge scale — a Mission Impossible delivered. |
There was a problem hiding this comment.
Casing is inconsistent for the same concept: this line introduces “GuardRails”, but later the doc refers to “automated guardrails”. Please pick one spelling/casing (typically “guardrails”) and use it consistently throughout the document to avoid confusion.
Suggested change
| Led an org of 350+ FTEs & Contractors as Project Lead at a major systems integrator, contracted to a global industrial conglomerate. Soon owned end-to-end Tier-0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech — then migration of 100s of AWS Accounts and Orgs across multiple business units (Energy & Nuclear, HealthCare, and Aerospace / Defense) with Commercial & Gov-Cloud AWS Accounts, NIST, Compliance, Cyber, GuardRails, and fully automated IaC at an XXLarge scale — a Mission Impossible delivered. | |
| Led an org of 350+ FTEs & Contractors as Project Lead at a major systems integrator, contracted to a global industrial conglomerate. Soon owned end-to-end Tier-0 Access, Control, & Authority, becoming a Trusted Advisor responsible for CoreTech — then migration of 100s of AWS Accounts and Orgs across multiple business units (Energy & Nuclear, HealthCare, and Aerospace / Defense) with Commercial & Gov-Cloud AWS Accounts, NIST, Compliance, Cyber, guardrails, and fully automated IaC at an XXLarge scale — a Mission Impossible delivered. |
|  | ||
|
|
||
| * **Min 0-10 (Containment):** Establish the blast radius. Is GE Healthcare or Vernova down? Find the common root cause. | ||
| * **Min 0-10 (Containment):** Establish the blast radius. Is the HealthCare BU or the Energy BU down? Find the common root cause. |
There was a problem hiding this comment.
This line uses “HealthCare” again. Please standardize the casing (e.g., “Healthcare BU”) consistently across the doc.
Suggested change
| * **Min 0-10 (Containment):** Establish the blast radius. Is the HealthCare BU or the Energy BU down? Find the common root cause. | |
| * **Min 0-10 (Containment):** Establish the blast radius. Is the Healthcare BU or the Energy BU down? Find the common root cause. |
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.
🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.