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.

Sign up for GitHub

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

Jump to bottom

chore: Move technical guides to the guide directory #3371

Merged
merged 9 commits into from
Feb 3, 2025
Merged

chore: Move technical guides to the guide directory #3371

merged 9 commits into from
Feb 3, 2025

Conversation

sfc-gh-jmichalak
Copy link
Contributor

@sfc-gh-jmichalak sfc-gh-jmichalak commented Jan 31, 2025 •
edited
Loading

  • Move technical guides to the guide directory.
  • Update the links to point to the new location.
  • Merge identifiers documents.
  • Replace a few links with relative links in the registry. This is done like links to the identifier guide in the secondary connection resource.
  • Update links to execute resource.
  • Add a note about assigning a resource monitor to account.

@github-actions GitHub Actions
Copy link

Integration tests cancelled for a8ee7affd7e8c403f02f7119f7c7cc9a192d9b83

sfc-gh-asawicki
sfc-gh-asawicki requested changes Feb 3, 2025
View reviewed changes
sfc-gh-jcieslak
sfc-gh-jcieslak approved these changes Feb 3, 2025
View reviewed changes
ROADMAP.md
@@ -19,7 +19,7 @@ We have released a long-awaited [v1.0.0](https://github.com/Snowflake-Labs/terra
Since the last update we have focused on:
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we also think about moving roadmap and migration_guide? (can be added later)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, I think we discussed it some time ago, and the downside is that we can't override such documents later and the users would see the specified version by default. If they browse docs with older versions, they won't see the latest update in the roadmap and migration guide. I think we even had an issue that one of the migration entries was incorrect, but we could fix it later.

Copy link
Contributor

Choose a reason for hiding this comment

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

As discussed, it's not in the scope for this PR.

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 3, 2025

Integration tests cancelled for 99ab555a54849cc401d21ff7bbcf39d61a0e7c01

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 3, 2025

Integration tests cancelled for fb2d21e33af576e7b9d7da4a2f971944217d76e0

@sfc-gh-asawicki sfc-gh-asawicki self-requested a review February 3, 2025 15:20
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 3, 2025

Integration tests cancelled for c02e938fc71af22f9834565a01e9a8110b34ac2a

@sfc-gh-jmichalak sfc-gh-jmichalak merged commit 2b95809 into dev Feb 3, 2025
8 of 9 checks passed
@sfc-gh-jmichalak sfc-gh-jmichalak deleted the technical-docs branch February 3, 2025 15:33
@snowflake-release-please snowflake-release-please bot mentioned this pull request Feb 5, 2025
Merged
sfc-gh-jcieslak pushed a commit that referenced this pull request Feb 5, 2025
@snowflake-release-please
chore(main): release 1.0.3 (#3380)
af5d1e4 
🤖 I have created a release *beep* *boop*
---


##
[1.0.3](v1.0.2...v1.0.3)
(2025-02-05)


### 🔧 **Misc**

* Add basic performance tests
([#3349](#3349))
([c57346b](c57346b))
* Add notes about managing grants on hybrid table
([#3368](#3368))
([e5e98bd](e5e98bd))
* Add preview and stable categories to resources and data sources in
docs
([#3370](#3370))
([b061434](b061434))
* Fix docs and update the protected users list
([#3365](#3365))
([f23e8cb](f23e8cb))
* Grant ownership common use cases
([#3356](#3356))
([97813b6](97813b6))
* Metric level fix with object renaming guide
([#3376](#3376))
([629ff92](629ff92))
* Move technical guides to the guide directory
([#3371](#3371))
([2b95809](2b95809))
* Prepare authentication methods guide
([#3364](#3364))
([c4a1c5f](c4a1c5f))
* Publish Performance tests summary
([#3359](#3359))
([d30d002](d30d002))
* Resolve issues and document account resource
([#3360](#3360))
([46b7a9d](46b7a9d))
* Upgrade tfplugindocs
([#3361](#3361))
([89f2b0a](89f2b0a))


### 🐛 **Bug fixes:**

* change EnforceNetworkRulesForInternalStages ddl to parameter
([#3343](#3343))
([beb01c7](beb01c7))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfc-gh-jcieslak sfc-gh-jcieslak sfc-gh-jcieslak approved these changes

@sfc-gh-asawicki sfc-gh-asawicki sfc-gh-asawicki approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sfc-gh-jmichalak @sfc-gh-jcieslak @sfc-gh-asawicki