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.

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

Cleanup and reorganization #54

Merged

Conversation

gpoo
Copy link
Contributor

@gpoo gpoo commented Sep 26, 2020

This is a series of patches that clean unused code, as well as reorganization
in the data directory, which ends up reducing the directories and Makefile.am
files to process. Also, there is renaming of files to make them consistent with
the rest of files.

I have another branch to work on top of this (or whatever changes were needed)
to use meson as a built system.

Make parameters consistent when using portable functions to build
a path.
No need to have many levels of directories for a few
files.
No need to have many levels of directories for a few files.
As we have one UI file in a gresource, it is better to have them
all.
Avoid distributing a single file in ui given that the
UI files are integrated into a gresource file.
README.md Outdated Show resolved Hide resolved
* Use limit of 80 columns. The idea is to make it easy to read
  either on a web browser, text editor, or terminal
* Remove HTML, use markdown formatting for image.
There are two README files. One is only required because of
autotools. Later we can migrate to meson, and the warning will
be gone. README.md already accomplish the same function.
@gpoo gpoo mentioned this pull request Sep 26, 2020
@mikelolasagasti mikelolasagasti merged commit d738024 into mikelolasagasti:master Sep 27, 2020
@mikelolasagasti mikelolasagasti added this to the 0.5.4 milestone Sep 27, 2020
@gpoo gpoo deleted the cleanup-and-reorganization branch September 27, 2020 12:29
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.

None yet

2 participants