Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 23, 2026

  • Created .github/agents/test-specialist.md custom agent definition
  • Implemented missing get_asset method in FlexMeasuresClient (bug fix)
  • Wrote comprehensive test for custom scheduler feature (test_trigger_schedule_with_custom_scheduler)
  • Updated test-specialist.md with project-specific testing patterns
  • Added pre-commit hooks instructions and linting guidance to test-specialist.md
  • All tests passing (29/29)
  • All pre-commit hooks passing

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 23, 2026 09:51
Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Copilot AI changed the title [WIP] Add custom scheduler selection feature Add test for custom scheduler feature and implement missing get_asset method Jan 23, 2026
Copilot AI requested a review from Flix6x January 23, 2026 09:57
…scheduler' into copilot/sub-pr-161

Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
@Flix6x Flix6x marked this pull request as ready for review January 23, 2026 13:58
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
@Flix6x
Copy link
Contributor

Flix6x commented Jan 23, 2026

@copilot update test-specialist.md with instructions to apply the .pre-commit-config.yaml and fix the linting issues on this PR (see latest workflow).

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 23, 2026

@copilot update test-specialist.md with instructions to apply the .pre-commit-config.yaml and fix the linting issues on this PR (see latest workflow).

Updated test-specialist.md with comprehensive pre-commit instructions in commit 2ef477b.

The guide now includes:

  • How to install and run pre-commit hooks
  • List of all configured hooks (black, isort, flake8, mypy, etc.)
  • Step-by-step instructions for fixing linting issues
  • Best practices for maintaining code quality

All pre-commit hooks now pass on this PR.

@Flix6x Flix6x changed the title Add test for custom scheduler feature and implement missing get_asset method Create test-specialist.md agent and test it out by adding a test for the custom-scheduler feature Jan 23, 2026
@Flix6x Flix6x merged commit 302dfa9 into feat/select-custom-scheduler Jan 23, 2026
10 checks passed
@Flix6x Flix6x deleted the copilot/sub-pr-161 branch January 23, 2026 14:15
Flix6x added a commit that referenced this pull request Jan 23, 2026
* feat: select custom scheduler

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: patch the 'custom-scheduler' asset attribute

Signed-off-by: F.N. Claessen <felix@seita.nl>

* style: black

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Create test-specialist.md agent and test it out by adding a test for the custom-scheduler feature (#162)

* Initial plan

* Create test-specialist custom agent

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* Add test for custom scheduler feature and implement get_asset method

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* feat: make sure we have an asset ID before fetching the asset

Signed-off-by: F.N. Claessen <felix@seita.nl>

* style: black

Signed-off-by: F.N. Claessen <felix@seita.nl>

* delete: duplicate method

Signed-off-by: F.N. Claessen <felix@seita.nl>

* style: black on tests

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: add pre-commit instructions to test-specialist.md

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

---------

Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
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