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

increase metadata ticker length #3086

Merged
merged 2 commits into from
Jan 21, 2022
Merged

increase metadata ticker length #3086

merged 2 commits into from
Jan 21, 2022

Conversation

piotr-iohk
Copy link
Contributor

@piotr-iohk piotr-iohk commented Jan 18, 2022

  • Iincrease metadata ticker length to 6 9 digits

Comments

Issue Number

ADP-1362

input-output-hk/offchain-metadata-tools#45

@piotr-iohk piotr-iohk requested a review from rvl January 18, 2022 20:25
@piotr-iohk piotr-iohk self-assigned this Jan 18, 2022
@rvl
Copy link
Contributor

rvl commented Jan 19, 2022

bors r+

iohk-bors bot added a commit that referenced this pull request Jan 19, 2022
3086: increase metadata ticker length r=rvl a=piotr-iohk



- [x] Iincrease metadata ticker length to 6 digits

### Comments

<!-- Additional comments, links, or screenshots to attach, if any. -->

### Issue Number
input-output-hk/offchain-metadata-tools#45


Co-authored-by: Piotr Stachyra <[email protected]>
Co-authored-by: Rodney Lorrimar <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 19, 2022

Build failed:

src/Test/Integration/Scenario/CLI/Shelley/Transactions.hs:392:59:
--
  | 1) CLI Specifications, SHELLEY_CLI_TRANSACTIONS, TRANS_ESTIMATE_09 - Invalid amount, 1.5
  | uncaught exception: IOException of type ResourceVanished
  | fd:164: hFlush: resource vanished (Broken pipe)

Probably:
#2855

@piotr-iohk
Copy link
Contributor Author

bors r+

iohk-bors bot added a commit that referenced this pull request Jan 19, 2022
3086: increase metadata ticker length r=piotr-iohk a=piotr-iohk


- [x] Iincrease metadata ticker length to 6 digits

### Comments

<!-- Additional comments, links, or screenshots to attach, if any. -->

### Issue Number

ADP-1362

input-output-hk/offchain-metadata-tools#45


Co-authored-by: Piotr Stachyra <[email protected]>
Co-authored-by: Rodney Lorrimar <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 19, 2022

Build failed:

src/Test/Integration/Scenario/CLI/Shelley/Transactions.hs:244:59:
--
  | 1) CLI Specifications, SHELLEY_CLI_TRANSACTIONS, TRANS_CREATE_06 - Invalid amount, -1000
  | uncaught exception: IOException of type ResourceVanished
  | fd:122: hFlush: resource vanished (Broken pipe)

Probably
#2855

@piotr-iohk
Copy link
Contributor Author

bors r+

iohk-bors bot added a commit that referenced this pull request Jan 20, 2022
3086: increase metadata ticker length r=piotr-iohk a=piotr-iohk


- [x] Iincrease metadata ticker length to 6 digits

### Comments

<!-- Additional comments, links, or screenshots to attach, if any. -->

### Issue Number

ADP-1362

input-output-hk/offchain-metadata-tools#45


Co-authored-by: Piotr Stachyra <[email protected]>
Co-authored-by: Rodney Lorrimar <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 20, 2022

Timed out.

@piotr-iohk
Copy link
Contributor Author

bors r+

iohk-bors bot added a commit that referenced this pull request Jan 20, 2022
3086: increase metadata ticker length r=piotr-iohk a=piotr-iohk

- [x] Iincrease metadata ticker length to ~6~ 9 digits

### Comments

<!-- Additional comments, links, or screenshots to attach, if any. -->

### Issue Number

ADP-1362

input-output-hk/offchain-metadata-tools#45


Co-authored-by: Piotr Stachyra <[email protected]>
Co-authored-by: Rodney Lorrimar <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 20, 2022

Timed out.

@piotr-iohk
Copy link
Contributor Author

bors r+

@rvl rvl force-pushed the piotr/increase-ticker-length branch from fd4000c to 63fec9e Compare January 21, 2022 09:27
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 21, 2022

Canceled.

@rvl
Copy link
Contributor

rvl commented Jan 21, 2022

I just rebased and pushed.

We need to update the token metadata unit tests because some ticker examples which were previously invalid are now valid.
I'll fix it now...

@rvl rvl force-pushed the piotr/increase-ticker-length branch from 63fec9e to 68e5f87 Compare January 21, 2022 09:42
@rvl
Copy link
Contributor

rvl commented Jan 21, 2022

Looking at the bors jobs above, the integration tests passed.
And the unit tests are passing for the PR branch build.
Since bors seems to be broken, I'll merge manually.

@rvl rvl merged commit f72734b into master Jan 21, 2022
@rvl rvl deleted the piotr/increase-ticker-length branch January 21, 2022 12:55
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.

2 participants