Skip to content

Conversation

pelikhan
Copy link
Contributor

@pelikhan pelikhan commented Aug 25, 2025

Always inline JavaScript instead of shared workflows. Creates self contained locked workflows.

  • Logic was moved to JavaScript files + test suite.
  • removing old templates

Copilot AI and others added 7 commits August 23, 2025 21:23
#22)

* Initial plan

* Replace check-team-member template with inlined JavaScript script

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

* Fix code formatting (remove trailing whitespace)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

* Add check_team_member.cjs to tsconfig.json and create comprehensive tests

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>
#24)

* Initial plan

* Reimplement add-reaction feature as inlined JavaScript

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

* Remove "remove" functionality from add-reaction feature, simplify to only support adding reactions

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

* Remove fallback and try/catch clauses from add-reaction feature

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review August 25, 2025 02:31
@pelikhan
Copy link
Contributor Author

test proxy fails on forks PRS, no secrets.

@pelikhan pelikhan merged commit 762c404 into githubnext:main Aug 25, 2025
5 of 6 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.

3 participants