Skip to content

Add CI workflow, Dependabot config, issue/PR templates - #6

Merged
Arun Sekhar (achandmsft) merged 2 commits into
mainfrom
add-ci-and-dependabot
May 28, 2026
Merged

Add CI workflow, Dependabot config, issue/PR templates#6
Arun Sekhar (achandmsft) merged 2 commits into
mainfrom
add-ci-and-dependabot

Conversation

@achandmsft

Copy link
Copy Markdown
Collaborator

Pull Request #6: CI workflow + Dependabot + issue/PR templates

What

Adds the validation surface area the repo needs before going public:

  • .github/workflows/validate.yml — runs on every PR + push to main. Four jobs in parallel:
    • bicep — installs Bicep CLI, runs bicep build + bicep lint on both main.bicep and foundry.bicep.
    • terraformterraform fmt -check -recursive, init -backend=false, and validate on infra-terraform/infra/.
    • python — installs requirements.txt and byte-compiles every sample script in src/.
    • shellshellcheck --severity=error on all *.sh (resolved via git ls-files) and PSScriptAnalyzer Error-level scan on all *.ps1.
  • .github/dependabot.yml — weekly dependency PRs for pip (root), GitHub Actions, and Terraform providers in infra-terraform/infra/. Each ecosystem capped at 5 open PRs with scoped labels and commit prefixes.
  • .github/ISSUE_TEMPLATE/bug_report.yml — structured bug-report form with required IaC variant / region / model / tool-versions / repro / expected / actual fields, plus required confirmations (searched duplicates, redacted IDs).
  • .github/ISSUE_TEMPLATE/feature_request.yml — problem / solution / alternatives / scope.
  • .github/ISSUE_TEMPLATE/config.yml — disables blank issues and pins three contact links (MSRC, Foundry docs, Claude SDK docs).
  • .github/PULL_REQUEST_TEMPLATE.md — what / why / how / verification checklist with the IaC-parity and no-secrets reminders.

Drive-by fix in this PR

  • terraform fmt -recursive on infra-terraform/infra/main.tf — a few local blocks had misaligned = columns from an older edit. Pure whitespace, no semantic change. Verified terraform validate still passes.

Verification

Ran each CI step locally against the current tree:

Step Result
terraform fmt -check -recursive ✅ exit 0 after the fmt fix in this PR
bicep build main.bicep + foundry.bicep ✅ exit 0 (3× BCP037 modelProviderData warnings — known type-metadata gap; the property is real and supported by the Cognitive Services RP)
python -m compileall -q src/ ✅ exit 0
PSScriptAnalyzer Error-level on all *.ps1 ✅ no error-level findings
shellcheck Not installed locally; will run in CI. Severity is error so cosmetic warnings won't break the build.

Non-changes

  • No README changes.
  • No new dependencies pulled into the runtime path.
  • No source / IaC changes other than the cosmetic tf fmt pass.
  • Repo metadata (visibility, description, topics, homepage, delete-branch-on-merge) was already set via the API in parallel with this PR.

Follow-ups (out of scope for this PR)

  • awesome-azd listing PR to Azure/awesome-azd.
  • Optional markdownlint-cli2 step on README + community files once we settle on a style config.

CI (.github/workflows/validate.yml) runs on PRs and pushes to main with 4 parallel jobs:

- bicep: build + lint both main.bicep and foundry.bicep

- terraform: fmt -check, init -backend=false, validate

- python: install requirements + compileall src/

- shell: shellcheck on *.sh (error severity) and PSScriptAnalyzer on *.ps1 (Error level)

Dependabot (.github/dependabot.yml): weekly bumps for pip, github-actions, and the terraform providers in infra-terraform/infra/. Each ecosystem capped at 5 open PRs with scoped labels and commit prefixes.

Issue templates (.github/ISSUE_TEMPLATE/*.yml): structured bug report (IaC variant / region / model / tool-versions / repro), feature request, and config.yml disabling blank issues with 3 contact links (MSRC, Foundry docs, Claude SDK docs).

PR template (.github/PULL_REQUEST_TEMPLATE.md): what/why/how/verification with IaC-parity and no-secrets checklist.

Drive-by: terraform fmt -recursive on infra-terraform/infra/main.tf fixes a few mis-aligned local-block columns from an older edit. Whitespace only, no semantic change.

All CI steps verified locally against the current tree.
@achandmsft
Arun Sekhar (achandmsft) merged commit 1e0c646 into main May 28, 2026
5 checks passed
@achandmsft
Arun Sekhar (achandmsft) deleted the add-ci-and-dependabot branch May 28, 2026 08:25
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.

1 participant