zlin / wgetpaste

MIT License
34 stars 18 forks source link

refactor and add ix.io pastebin #18

Closed nvinson closed 4 years ago

nvinson commented 4 years ago

Yes. I pushed a fix for it just now.

On 10/5/20 09:44, Bruno Cattáneo wrote:

@bcattaneo commented on this pull request.


In wgetpaste https://github.com/zlin/wgetpaste/pull/18#discussion_r499609011:

@@ -87,18 +120,19 @@ REGEX_URL_gists='s|^."html_url": "([^"]+gist[^"]+)".$|\1|p' REGEX_RAW_gists='s|^(https://gist.github.com\)\(/.*\)$|\1/raw\2|' escape_description_gists() { sed -e 's|"|\"|g' -e 's|\x1b|\u001b|g' -e 's|\r||g' <<< "$"; } escape_input_gists() { sed -e 's|\|\\|g' -e 's|\x1b|\u001b|g' -e 's|\r||g' -e 's|\t|\t|g' -e 's|"|\"|g' -e 's|$|\n|' <<< "$" | tr -d '\n'; } -json_gists() {

  • local description="${1}" language="${2}" content="${3}"
  • [[ "$language" = auto ]] && language="" || language=".$language"
  • echo "{\"description\":\"${description}\",\"public\":\"${PUBLIC_gists}\",\"files\":{\"${description//\/}${language}\":{\"content\":\"${content}\"}}" +POST_gists() {
  • local description="${2}" language="${3}" content="${6}"
  • [[ "$language" = auto ]] && language="" || language=".$language"
  • echo "{\"description\":\"${description}\",\"public\":\"${PUBLIC_gists}\",\"files\":{\"${description//\/}${language}\":{\"content\":\"${content}\"}}"
  • echo "{\"description\":\"${description}\",\"public\":\"${PUBLIC_gists}\",\"files\":{\"${description//\/}${language}\":{\"content\":\"${content}\"}}"

Duplicate |echo|?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/zlin/wgetpaste/pull/18#pullrequestreview-502070168, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVRMCFBUHCF3UL3TWWWTNLSJHETRANCNFSM4SEVZXWQ.

Cogitri commented 4 years ago

Please squash the changes of the "fix formatting" commit into the commit which actually introduced the tab indentation

nvinson commented 4 years ago

Squashed the commit.

Cogitri commented 4 years ago

Thanks!