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

支持多种配置文件类型 #36

Open
2 tasks
longfangsong opened this issue Jul 29, 2020 · 0 comments · Fixed by #77
Open
2 tasks

支持多种配置文件类型 #36

longfangsong opened this issue Jul 29, 2020 · 0 comments · Fixed by #77
Labels
easy This issue is easy to fix. feature request A new feature need to be implemented. good first issue Good for newcomers low priority This is not so important and can be implemented if you have time.

Comments

@longfangsong
Copy link
Collaborator

longfangsong commented Jul 29, 2020 •
edited
Loading

Describe the solution you'd like
除了已经支持的 yaml 还支持

  • toml
  • json

等

Additional context
不考虑支持xml，xml用来写配置文件是垃圾
@longfangsong longfangsong added the feature request A new feature need to be implemented. label Jul 29, 2020
@zhoudian64 zhoudian64 added the good first issue Good for newcomers label Jul 29, 2020
@longfangsong longfangsong added low priority This is not so important and can be implemented if you have time. easy This issue is easy to fix. labels Jul 29, 2020
takemorenap added a commit to takemorenap/autoAPI that referenced this issue Aug 3, 2020
@takemorenap
add a support to json SHUReeducation#36
1e17ebb 
add a support to json
now the cli tool can generate target dir by json, and it will choose the load function by the extension of the file assigned by '-f'.
@takemorenap takemorenap mentioned this issue Aug 3, 2020
Merged
longfangsong pushed a commit that referenced this issue Aug 3, 2020
@takemorenap
添加了对.json配置文件的支持 (#49)
6e6019b 
* add CLI flags  --force(incomplete)#31

add a flag --force ,but it wont change anything about Action

* Revert "add CLI flags  --force(incomplete)#31"

This reverts commit da2b0a0.

* Revert "Revert "add CLI flags  --force(incomplete)#31""

This reverts commit b48f9f6.

* Revert "add CLI flags  --force(incomplete)#31"

This reverts commit da2b0a0.

* bug fixed

when given 'output path' not existing will make finfo := nil and  cause runtime error

* add a support to json #36

add a support to json
now the cli tool can generate target dir by json, and it will choose the load function by the extension of the file assigned by '-f'.

* format

* resolved some problem
@laojianzi laojianzi mentioned this issue Sep 2, 2020
Merged
@longfangsong longfangsong reopened this Sep 6, 2021
@longfangsong longfangsong modified the milestone: v1.1 Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
easy This issue is easy to fix. feature request A new feature need to be implemented. good first issue Good for newcomers low priority This is not so important and can be implemented if you have time.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add toml support for config read laojianzi/autoAPI
2 participants
@longfangsong @zhoudian64