Closed yarikoptic closed 5 years ago
@yarikoptic I have updated the fontello font, but I can't push to your branch. What is your preferred workflow? Do you want me to PR your PR? Do you want to grant me permissions? Push my own commit to master after merging yours?
Let me know. :-)
---Alex
Also, as FYI for updating fontello, I just simply do a manual process (though there are programmatic methods, I've just never bothered yet). 1) go to fontello.com 2) "import" the config.json file in theme/static/fonts/ 3) search and add whatever glyph is needed 4) download and extract 5) overwrite fontello.woff, fontello.woff2, and config.json 6) port the added rule(s) from css/fontello.css to style.css
ontello font, but I can't push to your branch
Strange... You should be able to ... Sorry for seemingly silly question - did you try?
╭─aqw@brick ~/git/www.datalad.org on yarikoptic-enh-youtube
╰─➤ git push yarikoptic yarikoptic-enh-youtube
Enumerating objects: 19, done.
Counting objects: 100% (19/19), done.
Delta compression using up to 4 threads
Compressing objects: 100% (9/9), done.
Writing objects: 100% (10/10), 11.04 KiB | 5.52 MiB/s, done.
Total 10 (delta 4), reused 0 (delta 0)
remote: Resolving deltas: 100% (4/4), completed with 4 local objects.
To github.com:yarikoptic/datalad.org
! [remote rejected] yarikoptic-enh-youtube -> yarikoptic-enh-youtube (permission denied)
error: failed to push some refs to 'git@github.com:yarikoptic/datalad.org'
╭─aqw@brick ~/git/www.datalad.org on yarikoptic-enh-youtube
╰─➤ git remote show yarikoptic
* remote yarikoptic
Fetch URL: git@github.com:yarikoptic/datalad.org
Push URL: git@github.com:yarikoptic/datalad.org
HEAD branch: master
Remote branches:
enh-datasets tracked
enh-heudi tracked
enh-index tracked
enh-search tracked
enh-widen tracked
enh-youtube new (next fetch will store in remotes/yarikoptic)
master tracked
nitpicking tracked
Local ref configured for 'git push':
master pushes to master (fast-forwardable)
FML. And now I see it. Sigh....
I was trying to push to the yarikoptic-enh-youtube
branch, and in fact should have been pushing to enh-youtube
.
Sorry about the noise. It's pushed now.
---Alex
edit: formatting
Ok. Build passes. Merging.
Thanks! sorry I've missed initial troubleshooting email, glad to see you figure it out.
Closes #37
TODOs:
@aqw please finish it up