Skip to content

inline js instead of shared actions#213

Merged
pelikhan merged 8 commits intogithub:mainfrom
githubnext:un-shared-actions
Aug 25, 2025
Merged

inline js instead of shared actions#213
pelikhan merged 8 commits intogithub:mainfrom
githubnext:un-shared-actions

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>
…c (#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 github:main Aug 25, 2025
5 of 6 checks passed
@pelikhan pelikhan deleted the un-shared-actions branch September 4, 2025 15:38
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