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

BloClick jq formatted #401

Closed
wants to merge 3 commits into from
Closed

BloClick jq formatted #401

wants to merge 3 commits into from

Conversation

ivandiazperez
Copy link
Contributor

Overview

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

Copy link
Member

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

I'm sorry this is annoying but can you use jq and not open the file in an editor again afterwards but submit it directly? (or use cat or vim or less to check before resubmitting.)

It looks like your editor reformats the file:

Hexdump of file:
00000000  7b 20 0a 20 20 22 62 6f  64 79 22 3a 20 7b 0a 20  |{ .  "body": {. |
00000010  20 20 20 22 6d 65 73 73  61 67 65 73 22 3a 20 5b  |   "messages": [|
00000020  0a 20 20 20 20 20 20 7b  20 0a 20 20 20 20 20 20  |.      { .      |
00000030  20 20 22 40 74 79 70 65  22 3a 20 22 2f 63 6f 73  |  "@type": "/cos|
00000040  6d 6f 73 2e 73 74 61 6b  69 6e 67 2e 76 31 62 65  |mos.staking.v1be|
00000050  74 61 31 2e 4d 73 67 43  72 65 61 74 65 56 61 6c  |ta1.MsgCreateVal|
00000060  69 64 61 74                                       |idat|
00000064
Hexdump of jq output:
00000000  7b 0a 20 20 22 62 6f 64  79 22 3a 20 7b 0a 20 20  |{.  "body": {.  |
00000010  20 20 22 6d 65 73 73 61  67 65 73 22 3a 20 5b 0a  |  "messages": [.|
00000020  20 20 20 20 20 20 7b 0a  20 20 20 20 20 20 20 20  |      {.        |
00000030  22 40 74 79 70 65 22 3a  20 22 2f 63 6f 73 6d 6f  |"@type": "/cosmo|
00000040  73 2e 73 74 61 6b 69 6e  67 2e 76 31 62 65 74 61  |s.staking.v1beta|
00000050  31 2e 4d 73 67 43 72 65  61 74 65 56 61 6c 69 64  |1.MsgCreateValid|
00000060  61 74 6f 72                                       |ator|
00000064

@ivandiazperez
Copy link
Contributor Author

I'm sorry this is annoying but can you use jq and not open the file in an editor again afterwards but submit it directly? (or use cat or vim or less to check before resubmitting.)

It looks like your editor reformats the file:

Hexdump of file:
00000000  7b 20 0a 20 20 22 62 6f  64 79 22 3a 20 7b 0a 20  |{ .  "body": {. |
00000010  20 20 20 22 6d 65 73 73  61 67 65 73 22 3a 20 5b  |   "messages": [|
00000020  0a 20 20 20 20 20 20 7b  20 0a 20 20 20 20 20 20  |.      { .      |
00000030  20 20 22 40 74 79 70 65  22 3a 20 22 2f 63 6f 73  |  "@type": "/cos|
00000040  6d 6f 73 2e 73 74 61 6b  69 6e 67 2e 76 31 62 65  |mos.staking.v1be|
00000050  74 61 31 2e 4d 73 67 43  72 65 61 74 65 56 61 6c  |ta1.MsgCreateVal|
00000060  69 64 61 74                                       |idat|
00000064
Hexdump of jq output:
00000000  7b 0a 20 20 22 62 6f 64  79 22 3a 20 7b 0a 20 20  |{.  "body": {.  |
00000010  20 20 22 6d 65 73 73 61  67 65 73 22 3a 20 5b 0a  |  "messages": [.|
00000020  20 20 20 20 20 20 7b 0a  20 20 20 20 20 20 20 20  |      {.        |
00000030  22 40 74 79 70 65 22 3a  20 22 2f 63 6f 73 6d 6f  |"@type": "/cosmo|
00000040  73 2e 73 74 61 6b 69 6e  67 2e 76 31 62 65 74 61  |s.staking.v1beta|
00000050  31 2e 4d 73 67 43 72 65  61 74 65 56 61 6c 69 64  |1.MsgCreateValid|
00000060  61 74 6f 72                                       |ator|
00000064

Yeah, sure, will close this PR and submit a new one. MacOS and its tricks... :)

@ivandiazperez
Copy link
Contributor Author

ivandiazperez commented Oct 23, 2023 via email

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

3 participants