Skip to content

Changes to align devcontainer with hack rules and paths - #476

Merged
Marco Weber (qxsch) merged 1 commit into
microsoft:mainfrom
dsanchor:main
Sep 2, 2026
Merged

Marco Weber (qxsch) merged 1 commit into
microsoft:mainfrom
dsanchor:main

Conversation

@dsanchor

@dsanchor dsanchor commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

This pull request updates the Azure Fraud Intelligence hack's dev container and documentation to streamline environment setup and clarify onboarding steps for users. The main changes include removing redundant environment scripts, improving the dev container configuration, and enhancing the challenge and walkthrough instructions for clarity and usability.

Dev Container and Environment Setup Improvements:

  • Updated the main dev container configuration (devcontainer.json) to set a consistent name, specify the workspace folder, configure remote environment variables, switch from postCreateCommand to postAttachCommand, and remove the now-unnecessary post-create.sh script. This simplifies the Codespace setup and ensures the correct environment is loaded on attach. [1] [2] [3] [4]
  • Removed the unused baseenv script, as all required environment variables are now set via other mechanisms.

Documentation and Challenge Instructions:

  • Updated challenge and walkthrough markdown files to remove references to sourcing baseenv, instructing users to only source hackenv, which reflects the new environment setup process. [1] [2] [3] [4] [5] [6]
  • Improved onboarding instructions in both challenges and walkthroughs to clarify the Azure and GitHub sign-in process, including explicit steps for forking the repository, selecting the correct dev container configuration, and initializing the environment. Additional screenshots and step-by-step guidance were added for clarity. [1] [2] [3] [4] [5] [6] [7] [8] [9]

These changes collectively make the setup process more robust and user-friendly, reducing potential confusion for new users and ensuring a smoother onboarding experience.

@qxsch
Marco Weber (qxsch) merged commit 829bcd9 into microsoft:main Sep 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants