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

docs: refresh to prefer 'run' and 'invoke' over 'elt', and 'utilities' over 'transformers' and 'orchestrators' #7301

Merged
merged 5 commits into from
Feb 13, 2023

Conversation

pnadolny13
Copy link
Contributor

I was primarily trying to update references to transformers/orchestrators and elt so we recommend utilities and run but ended up refreshing a bunch of other stuff too:

  • Update transformers + orchestrators to utilities. I think I made it clear that everything is still supported but that we're transitioning to calling everything utilities

  • Prefer meltano run over meltano elt. Our main examples used elt so we were funneling new users towards that command, prefer run moving forward and elt is for existing users or specific use cases where they need it right now.

  • Replicate data page (integrations)

    • I added examples for what settings metadata should look like for replication method and id. Personally I always have trouble finding examples and end up back in the getting started guide.
    • Reorganized to move elt specific stuff down further on the page
  • I also cleaned up anything I found along the way including:

    • rewording some stuff related to state merging
    • a few typos
    • updated outdated container name and version references

@netlify
Copy link

netlify bot commented Feb 10, 2023

Deploy Preview for meltano ready!

Name Link
🔨 Latest commit 6e944bf
🔍 Latest deploy log https://app.netlify.com/sites/meltano/deploys/63e6c5ff8f10d5000871b61b
😎 Deploy Preview https://deploy-preview-7301--meltano.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.

@pnadolny13 pnadolny13 changed the title docs refresh docs: docs refresh Feb 10, 2023
Copy link
Contributor

@aaronsteers aaronsteers left a comment

Choose a reason for hiding this comment

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

Fantastic updates, @pnadolny13. I reviewed the full text and noted just one suggested change.

docs/src/_guide/configuration.md Outdated Show resolved Hide resolved
@aaronsteers aaronsteers changed the title docs: docs refresh docs: refresh to prefer 'run' and 'invoke' over 'elt', and 'utilities' over 'transformers' and 'orchestrators' Feb 10, 2023
docs/src/_concepts/plugins.md Show resolved Hide resolved
docs/src/_concepts/project.md Outdated Show resolved Hide resolved
Co-authored-by: Taylor A. Murphy <taylor@meltano.com>
@pnadolny13
Copy link
Contributor Author

@aaronsteers let me know if you have any additional feedback on this.

@aaronsteers aaronsteers dismissed their stale review February 13, 2023 16:15

Feedback addressed

Copy link
Contributor

@aaronsteers aaronsteers left a comment

Choose a reason for hiding this comment

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

👍

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.

None yet

4 participants