Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

Migrate configuration files to YAML #168

Open
1 of 4 tasks
dzhoshkun opened this issue Jul 24, 2018 · 17 comments
Open
1 of 4 tasks

Migrate configuration files to YAML #168

dzhoshkun opened this issue Jul 24, 2018 · 17 comments

Comments

@dzhoshkun
Copy link
Collaborator

dzhoshkun commented Jul 24, 2018

This is a sub-task of #166

For a roadmap, see the following comment: #166 (comment)

Basically:

@dzhoshkun
Copy link
Collaborator Author

dzhoshkun commented Jul 24, 2018

  • modify the CI script to read (only?) YAML files

@dzhoshkun
Copy link
Collaborator Author

Deprecation warnings are ignored by default, hence no user output in the above build. Apparently this decision has been (and continues to be) strongly debated.

@dzhoshkun
Copy link
Collaborator Author

dzhoshkun commented Jul 30, 2018

@dzhoshkun
Copy link
Collaborator Author

dzhoshkun commented Aug 6, 2018

  • compare new YAML files on this branch to INI files on dev branch before merging! (by reverting 9666eb9)

@dzhoshkun
Copy link
Collaborator Author

dzhoshkun commented Aug 6, 2018

  • remove INI files?

@dzhoshkun
Copy link
Collaborator Author

  • default CI sections do not load any configuration files, should test these changes using the slow tests etc.

@dzhoshkun
Copy link
Collaborator Author

dzhoshkun commented Aug 6, 2018

  • the last commit fails because there are hard-coded filenames featuring the .ini extension. Those should be replaced, possibly using a global file extension constant

@dzhoshkun
Copy link
Collaborator Author

  • references to / examples of INI files in documentation should be revised as well

@dzhoshkun
Copy link
Collaborator Author

@dzhoshkun
Copy link
Collaborator Author

dzhoshkun commented Aug 10, 2018

dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
…ms_same, as it's essentially accomplishing that
dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
dzhoshkun pushed a commit that referenced this issue Aug 20, 2018
dzhoshkun added a commit that referenced this issue Aug 20, 2018
…ll, to make it a 'new-style' class (Python 2)"

This reverts commit 85bee88.
dzhoshkun added a commit that referenced this issue Aug 20, 2018
dzhoshkun added a commit that referenced this issue Aug 20, 2018
…, as it was causing problems when calling super method
@dzhoshkun
Copy link
Collaborator Author

  • overridden methods with default parameter values cause problems (see above commit, and also fix a369e3e if applicable)

dzhoshkun added a commit that referenced this issue Sep 5, 2018
…ist comparison, which seems to fail due to different ordering
@dzhoshkun
Copy link
Collaborator Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants