Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Support XDG compatible paths #62

Closed
krisgesling opened this issue Nov 2, 2021 · 1 comment · Fixed by #63
Closed

Support XDG compatible paths #62

krisgesling opened this issue Nov 2, 2021 · 1 comment · Fixed by #63
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@krisgesling
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Mycroft is moving to be XDG compatible. The PR to move Skills is here: MycroftAI/mycroft-core#2803

Doing this would require an update for MSK to use the new XDG paths for at least:

  • Skill creation
  • Test creation
  • ? something else ?

Describe the solution you'd like
Use XDG file paths consistent with MycroftAI/mycroft-core#2803
@krisgesling krisgesling added enhancement New feature or request help wanted Extra attention is needed labels Nov 2, 2021
@krisgesling krisgesling mentioned this issue Nov 2, 2021
Closed
@PureTryOut PureTryOut mentioned this issue Nov 4, 2021
Merged
5 tasks
@PureTryOut
Copy link
Contributor

From what I can see msk create-test requires the user to specify the path to the skills, which means this needs no adaption for the XDG Base Directory specification no? The user just needs to provide the correct path themselves.

msk create has been changed in #63.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support XDG Base Directory compatible MSM
2 participants
@PureTryOut @krisgesling