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

chore: sync with osx-plugin-template-hardhat develop branch #5

Merged
merged 134 commits into from
Mar 15, 2024

Conversation

heueristik
Copy link

@heueristik heueristik commented Mar 15, 2024

clauBv23 and others added 21 commits March 12, 2024 09:34
Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com>
Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com>
…etwork-configs

feat: use new osx common network configs
* doc: added missing comment

* build: remove redundant dependency

* feat: add a function to generate a random ens, set the ens to the plugin settings

* feat: bump commons config version in subgraph

* feat: add random name function

* refactor: settings

---------

Co-authored-by: Claudia <claudiabarcelovaldes40@gmail.com>
…o-run-over-pr-intead-of-commits

fix: GitHub actions to run over PR changes instead of commit changes
@heueristik heueristik changed the title chore: sync with osx-plugin-template-hardhat the develop branch chore: sync with osx-plugin-template-hardhat develop branch Mar 15, 2024
@heueristik heueristik merged commit 851fc4d into develop Mar 15, 2024
6 of 7 checks passed
@heueristik heueristik deleted the feature/template-sync branch March 15, 2024 09:47
Comment on lines +254 to +264
export function generateRandomName(length: number): string {
const allowedCharacters = 'abcdefghijklmnopqrstuvwxyz-0123456789';
let result = '';
for (let i = 0; i < length; i++) {
result += allowedCharacters.charAt(
Math.floor(Math.random() * allowedCharacters.length)
);
}
return result;
}

Copy link

@clauBv23 clauBv23 Mar 15, 2024

Choose a reason for hiding this comment

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

This function should not be needed in the multisig, it was used for the template plugin name, but here it is always the same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
8 participants