Skip to content

Conversation

@WSHAPER
Copy link

@WSHAPER WSHAPER commented Oct 11, 2025

  • Consolidate and emphasize SKILL.md usage patterns
  • Reorganize file handling and creation rules
  • Clarify file sharing and output strategy
  • Add missing sections to system message

…ng added function definitions

- Consolidate and emphasize SKILL.md usage patterns
- Reorganize file handling and creation rules
- Clarify file sharing and output strategy
- Add missing sections to system message
@asgeirtj
Copy link
Owner

Updated

@asgeirtj asgeirtj closed this Oct 12, 2025
@asgeirtj
Copy link
Owner

Done

@WSHAPER
Copy link
Author

WSHAPER commented Oct 12, 2025

Hi @asgeirtj,

I see this last commit's diff looks like an entire copy/replacement, which makes the commit history and context hard to follow.

It looks like something has confused Git. Would it be possible to revert that change and instead properly merge or cherry-pick the relevant commit?

@asgeirtj asgeirtj reopened this Oct 12, 2025
@asgeirtj
Copy link
Owner

I realized that a lot of stuff was missing so I redid the file with some humanreadable edits also did your commit include anything that is still missing?

@WSHAPER
Copy link
Author

WSHAPER commented Oct 13, 2025

Hi @asgeirtj,

Thanks for the update! Is there an issue with checking out the repo, reverting the last commit, and merging the original changes properly (e.g. via the green "Merge" button)? That should keep the history clean and avoid the full file replacement.

[…] also did your commit include anything that is still missing?

Sorry, I’m not sure I fully follow your comment. I won’t be going through the changes manually. The initial pull request already includes all the necessary changes in the diff. Is there a problem with reverting the previous commit and merging?

Copy link

@Daytona39264 Daytona39264 left a comment

Choose a reason for hiding this comment

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

Git

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