Skip to content

docs(transformer/class-properties): document transform options#7478

Merged
graphite-app[bot] merged 1 commit intomainfrom
11-25-docs_transformer_class-properties_document_transform_options
Nov 25, 2024
Merged

docs(transformer/class-properties): document transform options#7478
graphite-app[bot] merged 1 commit intomainfrom
11-25-docs_transformer_class-properties_document_transform_options

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Nov 25, 2024

Add docs for the loose option / set_public_class_fields assumption.

@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 25, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

Copy link
Member Author

overlookmotel commented Nov 25, 2024

@github-actions github-actions bot added A-transformer Area - Transformer / Transpiler C-docs Category - Documentation. Related to user-facing or internal documentation labels Nov 25, 2024
@overlookmotel overlookmotel marked this pull request as ready for review November 25, 2024 16:42
@overlookmotel
Copy link
Member Author

Just docs, so merging without review.

@overlookmotel overlookmotel added the 0-merge Merge with Graphite Merge Queue label Nov 25, 2024
Copy link
Member Author

overlookmotel commented Nov 25, 2024

Merge activity

  • Nov 25, 11:45 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 25, 11:51 AM EST: The Graphite merge queue couldn't merge this PR because it had conflicts with the trunk branch.
  • Nov 25, 11:51 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 25, 4:51 PM UTC: The merge label '0-merge' was removed. This PR will no longer be merged by the Graphite merge queue
  • Nov 25, 11:52 AM EST: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • Nov 25, 12:17 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 25, 12:17 PM EST: A user added this pull request to the Graphite merge queue.
  • Nov 25, 12:22 PM EST: A user merged this pull request with the Graphite merge queue.

@overlookmotel overlookmotel changed the base branch from 11-25-refactor_transformer_class-properties_rename_var to graphite-base/7478 November 25, 2024 16:45
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 25, 2024

CodSpeed Performance Report

Merging #7478 will not alter performance

Comparing 11-25-docs_transformer_class-properties_document_transform_options (2a5954a) with main (abb0e0e)

Summary

✅ 30 untouched benchmarks

@graphite-app graphite-app bot changed the base branch from graphite-base/7478 to main November 25, 2024 16:50
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Nov 25, 2024
@overlookmotel overlookmotel force-pushed the 11-25-docs_transformer_class-properties_document_transform_options branch from c063298 to 82eb5be Compare November 25, 2024 16:51
@overlookmotel overlookmotel added the 0-merge Merge with Graphite Merge Queue label Nov 25, 2024
Add docs for the `loose` option / `set_public_class_fields` assumption.
@overlookmotel overlookmotel force-pushed the 11-25-docs_transformer_class-properties_document_transform_options branch from 82eb5be to 2a5954a Compare November 25, 2024 17:17
@graphite-app graphite-app bot merged commit 2a5954a into main Nov 25, 2024
@graphite-app graphite-app bot deleted the 11-25-docs_transformer_class-properties_document_transform_options branch November 25, 2024 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0-merge Merge with Graphite Merge Queue A-transformer Area - Transformer / Transpiler C-docs Category - Documentation. Related to user-facing or internal documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant