eexpress / gs-git

monitor git directory for changes.
GNU General Public License v3.0
8 stars 1 forks source link

fixes some stuff from #2 #3

Closed mipmip closed 1 year ago

mipmip commented 1 year ago
eexpress commented 1 year ago

After I do this.

git checkout -b mipmip-main main
git config pull.rebase true
git pull https://github.com/mipmip/gs-git.git main

I meet this error when ./install local-install.

data>:13:1: Parse error: unexpected character `<', expected character `]'

which comes from this line.

gnome-extensions pack src \
    --force \
    --extra-source="../LICENSE" \
    --extra-source="git-monitor.json" \
    --extra-source="ui.js" \
    --extra-source="org.gnome.gitg-symbolic.svg" \
    --extra-source="org.gnome.gitg-symbolic-alert.svg" \
    --extra-source="../CHANGELOG.md"

I don't know why.

mipmip commented 1 year ago

@eexpress It is the malformed metadata.json. If you tidy this file it will work.

eexpress commented 1 year ago

Sorry for long AFK. I read your code. It seems "Alert dirty repos" has not effect in code, I'm not sure what feature you want. "Show changed files" is same here.

I also want redefine those actions when right/left click on dir/file menuitems. So I will use "git difftool -d" with dirs, use "git difftool" with files and all set to button 1. Then use system open action on button 3. So we don't need prefs.js anymore.

Due to "Separate" directory settings in config file. It is not so important.

Thanks for your PR, but I like more simple one. I think you can make your own extension and commit to https://extensions.gnome.org/. Your code is more structured and rational than my.

mipmip commented 1 year ago

Thanks for your reply. This PR was to big in the first place. I will fork this project and I'm very thankful for the inspiration you gave me.