Skip to content

Conversation

@aholstrup1
Copy link
Collaborator

❔What, Why & How

Update docs with instructions for tests as requested in office hours

✅ Checklist

  • Add tests (E2E, unit tests)
  • Update RELEASENOTES.md
  • Update documentation (e.g. for new settings or scenarios)
  • Add telemetry

Copilot AI and others added 2 commits October 29, 2025 10:04
* Initial plan

* Add comprehensive testing notes to Contribute.md

Co-authored-by: aholstrup1 <117829001+aholstrup1@users.noreply.github.com>

* Remove E2E testing documentation, keep only unit tests

Co-authored-by: aholstrup1 <117829001+aholstrup1@users.noreply.github.com>

* Update unit test best practices

Removed the guideline to follow the Arrange-Act-Assert pattern from best practices for unit tests.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: aholstrup1 <117829001+aholstrup1@users.noreply.github.com>
@aholstrup1 aholstrup1 requested a review from a team as a code owner October 29, 2025 10:14
Copilot AI review requested due to automatic review settings October 29, 2025 10:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the contribution documentation for AL-Go by adding comprehensive guidance on testing practices and improving the GitHub Codespaces section. The changes aim to help contributors better understand when and how to write unit tests for their contributions.

Key Changes:

  • Added a new "Testing your contributions" section with guidelines on when to add tests
  • Expanded the "Unit tests" section with detailed instructions, examples, and best practices for writing Pester tests
  • Updated the GitHub Codespaces section with a quick-start badge and streamlined instructions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@aholstrup1 aholstrup1 enabled auto-merge (squash) October 29, 2025 10:45
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.

3 participants