Skip to content

Adding auto formatting#8

Merged
ananas-block merged 3 commits intodev_v4from
swen-dev_v4-adding-auto-formatting
Jan 15, 2023
Merged

Adding auto formatting#8
ananas-block merged 3 commits intodev_v4from
swen-dev_v4-adding-auto-formatting

Conversation

@SwenSchaeferjohann
Copy link
Contributor

removes future merge conflict headaches by enforcing a shared formatting style & auto-formatting (for light-sdk-ts)

  • uses husky to manage git hooks
  • adds pre-commit git hook to auto-format on commit
  • adds shared .prettierrc config to enforce style

similar should probably be done for the rust code

the current style is open for discussion:

{
"trailingComma": "all",
"tabWidth": 2,
"singleQuote": false
}

@ananas-block ananas-block merged commit 56a2ba5 into dev_v4 Jan 15, 2023
@SwenSchaeferjohann SwenSchaeferjohann deleted the swen-dev_v4-adding-auto-formatting branch January 15, 2023 04:37
vadorovsky referenced this pull request in vadorovsky/light-protocol May 1, 2024
ananas-block added a commit that referenced this pull request Feb 16, 2026
- create.rs: verify non-compressible token account is rent-exempt before
  initializing (audit issue #8)
- create_ata.rs: in idempotent mode, deserialize existing account and
  verify mint and owner fields match expected values (audit issue #4)

Entire-Checkpoint: caaa14ac3051
ananas-block added a commit that referenced this pull request Feb 17, 2026
…oken account check, add additiona create ata idempotent check (#2292)

* fix: add rent-exemption and mint/owner checks for token account creation

- create.rs: verify non-compressible token account is rent-exempt before
  initializing (audit issue #8)
- create_ata.rs: in idempotent mode, deserialize existing account and
  verify mint and owner fields match expected values (audit issue #4)

Entire-Checkpoint: caaa14ac3051

* fix:

Entire-Checkpoint: caaa14ac3051

* add tests and format

Entire-Checkpoint: 2b4028368dbf

* chore: restore photon submodule to match main
Entire-Checkpoint: c298aaf24a18
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