Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Clarify scope #21

Closed
jcbhmr opened this issue Mar 10, 2023 · 7 comments
Closed

Clarify scope #21

jcbhmr opened this issue Mar 10, 2023 · 7 comments
Assignees
Labels
documentation Improvements or additions to documentation question Further information is requested

Comments

@jcbhmr
Copy link
Collaborator

jcbhmr commented Mar 10, 2023

I think we should clarify whether this action should include things like preprocessing links, dealing with uncommitted changes, renaming README.md ➡ Home.md etc.

Open questions

  • If I want to rename README.md to Home.md in my wiki, does that belong here or do i need to manually mv wiki/README.md wiki/Home.md in my workflow?
  • If I want to change all markdown links to strip off the .md suffix so they work on the wiki, is that in scope here, or should that be done in a preprocess step like Andrew-Chen-Wang/strip-md-suffix-markdown-links or similar?

/assign jcbhmr

Originally posted by @jcbhmr in #9 (comment)

@Andrew-Chen-Wang are there any other scope questions that you want addressed/documented/discussed?

@jcbhmr
Copy link
Collaborator Author

jcbhmr commented Mar 10, 2023

in my opinion which I am perfectly OK with being overruled is that these are both out of scope and should be encouraged to:

- uses: actions/checkout
- uses: user/strip-md-links
  with:
    path: wiki
- run: git add -A && git commit --amend --no-edit
- uses: Andrew-Chen-Wang/github-wiki-action

or similar

@jcbhmr
Copy link
Collaborator Author

jcbhmr commented Mar 10, 2023

You know I'm just now realizing how much I have just interjected myself forcefully into this project... 🤔 is this upheaval OK with you @Andrew-Chen-Wang or would you like my to pump the brakes a bit?

@Andrew-Chen-Wang
Copy link
Owner

happy to make you a co maintainer. I'll just add in a review requirement before merging for both of us. appreciate it as work has taken a toll on me which meant less OSS work!

@Andrew-Chen-Wang
Copy link
Owner

If I want to rename README.md to Home.md in my wiki, does that belong here or do i need to manually mv wiki/README.md wiki/Home.md in my workflow?

Should be a flag default True.

If I want to change all markdown links to strip off the .md suffix so they work on the wiki, is that in scope here, or should that be done in a preprocess step like Andrew-Chen-Wang/strip-md-suffix-markdown-links or similar?

Not sure what you mean here.

  • run: git add -A && git commit --amend --no-edit

Not fully opposed to pre-processing actions, but we want to do as much as possible inside the script and give users an option/flag unless it's not possible. Also strive to keep backwards compatibility.

@jcbhmr
Copy link
Collaborator Author

jcbhmr commented Mar 10, 2023

If I want to change all markdown links to strip off the .md suffix so they work on the wiki, is that in scope here, or should that be done in a preprocess step like Andrew-Chen-Wang/strip-md-suffix-markdown-links or similar?

Not sure what you mean here.

What I'm talking about is the problem discussed in #8 where a link like [View our help page](./Help.md) works only when viewing it in the GitHub source tree but them if you just push that file to the wiki, the .md suffix won't work because links are no longer https://github.com/Andrew-Chen-Wang/github-wiki-action/blob/master/README.md, they are https://github.com/Andrew-Chen-Wang/github-wiki-action/wiki/Test-Page-1

Note that the README.md is README.md, not just README as it would be when deployed to the wiki itself.

If I want to change all markdown links to strip off the .md suffix so they work on the wiki, is that in scope here, or should that be done in a preprocess step like Andrew-Chen-Wang/strip-md-suffix-markdown-links or similar?

What I'm getting at here is: which of these do you want? Do we want to offer that feature or do we want it to be a preprocess step done by some other action?

- uses: user/strip-md-links@v1
- uses: EndBug/add-and-commit@v9
- uses: Andrew-Chen-Wang/github-wiki-action@v4
- uses: Andrew-Chen-Wang/github-wiki-action@v4

@jcbhmr
Copy link
Collaborator Author

jcbhmr commented Mar 10, 2023

Not fully opposed to pre-processing actions, but we want to do as much as possible inside the script and give users an option/flag unless it's not possible. Also strive to keep backwards compatibility.

Awesome ❤ thanks for the vision clarification.

Here's some ideas of a list of features to tout on the README:

  • We auto-rename README.md to Home.md
  • We auto-strip .md from link suffixes
  • We support auto commit of changes (default true or false?)

@jcbhmr
Copy link
Collaborator Author

jcbhmr commented Mar 10, 2023

happy to make you a co maintainer. I'll just add in a review requirement before merging for both of us. appreciate it as work has taken a toll on me which meant less OSS work!

Cool! 👍 I'll try to be somewhat cautious and incremental with my changes and PRs.

@jcbhmr jcbhmr self-assigned this Mar 11, 2023
@jcbhmr jcbhmr added documentation Improvements or additions to documentation question Further information is requested labels Mar 11, 2023
Repository owner locked and limited conversation to collaborators Mar 14, 2023
@jcbhmr jcbhmr converted this issue into discussion #26 Mar 14, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
documentation Improvements or additions to documentation question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants