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

BRUSHIZER COLLECTION #18

Closed
wants to merge 4 commits into from
Closed

BRUSHIZER COLLECTION #18

wants to merge 4 commits into from

Conversation

Mezange
Copy link
Contributor

@Mezange Mezange commented Jun 18, 2024

No description provided.

@ifavo
Copy link
Collaborator

ifavo commented Jun 25, 2024

@Mezange Can you please check the failed build?

It requires the contracts folder to be all lower case:

path: /home/runner/work/nft-registry/nft-registry/tokens/main/0xba2834c43670a28175F965Be544386D04B602243
message: "folder name should be a valid address with lower case"

Can you please adjust? :-)

@Mezange
Copy link
Contributor Author

Mezange commented Jun 27, 2024

Hi my friend :)

I don't know if the commit went through... can you put everything in lowercase for me if my commit didn't go through ?

Everything is good ?

@ifavo
Copy link
Collaborator

ifavo commented Jun 28, 2024

@Mezange it still contains upper case chars. Likely because you just renamed and comitted locally, I suggest to:

  1. rename
  2. commit
  3. rename to lower case
  4. commit again

Otherwise the case-insensitivity in some filesystems can mess this up sometimes. Or you just fork a again in a fresh repo and add the folder in lower case.

@Mezange
Copy link
Contributor Author

Mezange commented Jun 28, 2024

I renamed the folder name but during commit it does not change.

What am I supposed to do? :/ lol

@ifavo
Copy link
Collaborator

ifavo commented Jun 28, 2024

I renamed the folder name but during commit it does not change.

What am I supposed to do? :/ lol

Did you rename it to something different like _x-something-else, so it won't be the same folder at the end?

You can also create a new branch of fork a new repo, and add it in lower case again.

@Mezange Mezange closed this by deleting the head repository Jun 28, 2024
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