Skip to content

refactor: remove profiler4fun#64

Merged
nicolasparada merged 3 commits into
mainfrom
ditch4fun
May 29, 2026
Merged

refactor: remove profiler4fun#64
nicolasparada merged 3 commits into
mainfrom
ditch4fun

Conversation

@nicolasparada
Copy link
Copy Markdown
Contributor

Remove profiler_4fun input option, and remove all references to such functionality from the action.

Copilot AI review requested due to automatic review settings May 29, 2026 02:39
Copy link
Copy Markdown

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

Removes the experimental profiler_4fun input and all associated code paths, simplifying the action to always use the standard profiler output. Also normalizes action.yaml string quoting to double-quotes.

Changes:

  • Drops profiler_4fun input from action.yaml and the related state/env wiring in src/main.js, src/action.js, and src/post.js.
  • Simplifies resolveSelectedProfilerFile() in src/post.js to a single, fixed lookup chain for the standard profiler file.
  • Rebuilds bundled dist/main/index.js and dist/post/index.js (including unrelated upstream dependency updates pulled in by the rebuild).

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
action.yaml Removes profiler_4fun input and reformats string quoting.
src/main.js Drops profiler4fun state/env saving.
src/action.js Removes JIBRIL_PROFILER4FUN_FILE from generated Jibril env file.
src/post.js Removes selected-profiler resolution; always uses standard profiler file.
dist/main/index.js Regenerated bundle reflecting source changes plus dependency updates.
dist/post/index.js Regenerated bundle reflecting source changes plus dependency updates.

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

@nicolasparada nicolasparada merged commit 46e1d38 into main May 29, 2026
4 checks passed
@nicolasparada nicolasparada deleted the ditch4fun branch May 29, 2026 03:19
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