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

New sanity check (Golang) #15960

Merged
merged 4 commits into from
Nov 22, 2021
Merged

New sanity check (Golang) #15960

merged 4 commits into from
Nov 22, 2021

Conversation

unanoc
Copy link
Contributor

@unanoc unanoc commented Nov 22, 2021

No description provided.

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
File not allowed: .github/assets.config.yaml. Please revert it.
File not allowed: .github/workflows/pr-ci.yml. Please revert it.
File not allowed: cmd/main.go. Please revert it.
File not allowed: go.mod. Please revert it.
File not allowed: go.sum. Please revert it.

@ghost
Copy link

ghost commented Nov 22, 2021

Hi! In order to compensate for the efforts of processing PRs, we kindly ask for a contribution.
💀 As there is no refund, before paying the fee, make sure new tokens fulfill the minimum circulation and other acceptance criteria.
See also the PR Fee FAQ.

Please pay 1000 TWT with the memo 15960 to the address bnb1tqq9llyr3dyjd559dha6z5r5etk3qfwk07m098.
Alternatively, 5 BNB is also accepted (same memo & address).

QR code: Trust | other wallet

Notes:

  • Trust Wallet Tokens (TWT) can be obtained through our Referral Program or from DEXs/exchanges.
  • New tokens without significant usage are rejected, see criteria mentioned above.
  • One PR should be for a single project; PR's with more than 10 logos will be rejected.
  • Payment evaluation happens automatically, but with a few minutes delay. When payment is detected, an Accept Review is automatically placed on the PR, which is a condition for merge.
  • Evaluating the PR is done manually, and it is merged only if all conditions are satisfied.
  • TWT-BEP2 is supported (Binance Chain), TWT-BEP20 version on Smart Chain is not.

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
File not allowed: .github/assets.config.yaml. Please revert it.
File not allowed: .github/workflows/pr-ci.yml. Please revert it.
File not allowed: cmd/main.go. Please revert it.
File not allowed: go.mod. Please revert it.
File not allowed: go.sum. Please revert it.

go.mod Outdated

require (
github.com/sirupsen/logrus v1.8.1
github.com/trustwallet/assets-go-libs v0.0.0-20211122144736-3715146de5f7
Copy link
Contributor

Choose a reason for hiding this comment

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

can you use tagged version instead?

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
File not allowed: .github/assets.config.yaml. Please revert it.
File not allowed: .github/workflows/pr-ci.yml. Please revert it.
File not allowed: cmd/main.go. Please revert it.
File not allowed: go.mod. Please revert it.
File not allowed: go.sum. Please revert it.

1 similar comment
@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
File not allowed: .github/assets.config.yaml. Please revert it.
File not allowed: .github/workflows/pr-ci.yml. Please revert it.
File not allowed: cmd/main.go. Please revert it.
File not allowed: go.mod. Please revert it.
File not allowed: go.sum. Please revert it.

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
Too many changed files, 98, max 20. If not all files are intended, check if forked repo is up to date.

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
Too many changed files, 99, max 20. If not all files are intended, check if forked repo is up to date.

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
Too many changed files, 99, max 20. If not all files are intended, check if forked repo is up to date.

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
Too many changed files, 99, max 20. If not all files are intended, check if forked repo is up to date.

@ghost
Copy link

ghost commented Nov 22, 2021

PR Summary
Too many changed files, 99, max 20. If not all files are intended, check if forked repo is up to date.

@unanoc unanoc merged commit 0b29b70 into master Nov 22, 2021
@unanoc unanoc deleted the daniel/new-ci branch November 22, 2021 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Payment Status: Paid Fee completely paid for PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants