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

Added possibility to pass command line arguments #91

Merged
merged 5 commits into from
Mar 17, 2017

Conversation

bchalls
Copy link
Contributor

@bchalls bchalls commented Mar 16, 2017

Added possibility to pass command line arguments to allow for batch yaml file upload

batchUpload.sh Outdated

subdomain=''
email=''
password=''
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can foresee someone editing this file and accidentally committing it exposing their details.

Perhaps call this file something like batchUploadExample.sh, ignore batchUpload.sh from Git, and instruct people to copy the former to the latter and edit that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. I have made the change and pushed it.

@zachfeldman
Copy link
Contributor

Almost there @bchalls and thanks for the PR! Would you mind just editing the docs to reflect that you have to copy the example file and rename it for this to work? Then I'll merge. Thanks for the review @hugovk !

@zachfeldman
Copy link
Contributor

Looking good, thanks @bchalls . Merging!

@zachfeldman zachfeldman merged commit 97836a7 into lambtron:master Mar 17, 2017
jordan8037310 added a commit to jordan8037310/emojipacks that referenced this pull request Jun 14, 2017
* commit '8f20d6d538ad6b58b7da085cef43237c73d928d8': (49 commits)
  Revert "created an all emoji file (lambtron#118)" (lambtron#119)
  created an all emoji file (lambtron#118)
  Added prefix to occupy emoji. [lambtron#116] (lambtron#116)
  Adds Political Pack of Emojis (lambtron#115)
  bump patch
  fix upload slackmoji yaml file
  add some more beasts to the Animals pack including new emoji :columbian-mammoth: 🦤 :emu: :platypus: :sabre-tooth-tiger: :wildebeest: :wooly-mammoth: 🦓 :arctic-fox: :desert-fox: :festive-reindeer: :great-auk: :great-owl: :hawk: :moa: 🫎 :reindeer-crash: :reindeer-harness: :beluga-whale: :blue-whale: :grey-seal: :humpback-whale: :killer-whale: :narwhal2: 🦝 :robot-dog: :robot-scorpion: :robot-shark: :coyote: (lambtron#111)
  Unicorn Pack (lambtron#110)
  New Weapons Pack. (lambtron#108)
  fix link (lambtron#109)
  Add gamedevmoji (lambtron#107)
  Allow emoji aliases to be created from emojipacks (lambtron#106)
  image-checker.py: Fix python 3 compatibility (lambtron#105)
  Add Shiba emojipack (lambtron#103)
  Add Marvel Avengers emojipack (lambtron#101)
  Added hamsterdance emojipack (lambtron#99)
  Version bump.
  Removed --harmony option from shebang (lambtron#96)
  Fixes broken links in skype.yaml v2 (lambtron#93)
  Added possibility to pass command line arguments (lambtron#91)
  ...
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.

3 participants