Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

refactor(rome_js_analyze): move useIframeTitle from semantic_analyzer to analyzer and refactoring #4438

Merged
merged 3 commits into from
May 4, 2023

Conversation

nissy-dev
Copy link
Contributor

@nissy-dev nissy-dev commented May 4, 2023

Summary

I moved useIframeTitle from semantic_analyzer to analyzer and refactored.

Test Plan

Changelog

This is an internal refactoring

  • The PR requires a changelog line

Documentation

  • The PR requires documentation
  • I will create a new PR to update the documentation

@netlify
Copy link

netlify bot commented May 4, 2023

Deploy Preview for docs-rometools ready!

Name Link
🔨 Latest commit 1575d6b
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/645373207774ca0008d0b521
😎 Deploy Preview https://deploy-preview-4438--docs-rometools.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added A-Linter Area: linter A-Project Area: project configuration and loading labels May 4, 2023
@nissy-dev nissy-dev changed the title refactor(rome_js_analyze): move useIframeTitle from semantic_analyzer to analyzer refactor(rome_js_analyze): move useIframeTitle from semantic_analyzer to analyzer and refactoring May 4, 2023
Copy link
Contributor

@ematipico ematipico left a comment

Choose a reason for hiding this comment

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

Amazing! 🚀

Please make sure to rebase and regenerate all the files! I leave the merge of this PR to you :)

@nissy-dev nissy-dev merged commit ed2473f into main May 4, 2023
@nissy-dev nissy-dev deleted the refactor/use-iframe-title branch May 4, 2023 09:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Linter Area: linter A-Project Area: project configuration and loading
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants