Skip to content

added Interview with Provenance Tags to the Developer Spotlight section + New page - Office Hours#1342

Merged
pablozsc merged 11 commits into
mainfrom
provenance-tags-interview
Apr 30, 2025
Merged

added Interview with Provenance Tags to the Developer Spotlight section + New page - Office Hours#1342
pablozsc merged 11 commits into
mainfrom
provenance-tags-interview

Conversation

@pablozsc

@pablozsc pablozsc commented Apr 11, 2025

Copy link
Copy Markdown
Contributor

Purpose

New article added - interview with Provenance Tags
Added new page - Office hours

Changes

  • fixed typos in AesirX interview article
  • each interview is now located in its own directory

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.

By submitting the contribution I accept the terms and conditions of the
Contributor License Agreement v1.0

@pablozsc pablozsc requested review from TinaKT and dragosgd April 11, 2025 14:30
@pablozsc pablozsc self-assigned this Apr 11, 2025
@pablozsc pablozsc added the preview Trigger a documentation preview for a PR. label Apr 11, 2025
@github-actions

github-actions Bot commented Apr 11, 2025

Copy link
Copy Markdown
PR Preview Action v1.6.1
Preview removed because the pull request was closed.
2025-04-30 14:28 UTC

@pablozsc pablozsc changed the title added Interview with Provenance Tags to the Developer Spotlight section added Interview with Provenance Tags to the Developer Spotlight section + New page - Office Hours Apr 18, 2025
@pablozsc pablozsc added preview Trigger a documentation preview for a PR. and removed preview Trigger a documentation preview for a PR. labels Apr 18, 2025
@dragosgd dragosgd added preview Trigger a documentation preview for a PR. and removed preview Trigger a documentation preview for a PR. labels Apr 23, 2025

@dragosgd dragosgd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Minor changes requested, looks great overall!

Comment thread source/mainnet/community/developer-spotlight/aesirx/index.rst Outdated
Comment thread source/mainnet/community/developer-spotlight/provenance-tags/index.rst Outdated
Comment thread source/mainnet/community/developer-spotlight/provenance-tags/index.rst Outdated
@dragosgd dragosgd self-requested a review April 23, 2025 09:56

@TinaKT TinaKT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks really good, only minor suggested changes.
One small tip: our style guide recommends lowercase words in headings except for the first word, proper nouns, and established terms (e.g. Office Hours). This is just our chosen convention. Great job overall!

Comment thread source/mainnet/community/developer-spotlight/provenance-tags/index.rst Outdated
Comment thread source/mainnet/community/developer-spotlight/provenance-tags/index.rst Outdated
Comment thread source/mainnet/community/office-hours/index.rst Outdated
Comment thread source/mainnet/community/office-hours/index.rst Outdated
Comment thread source/mainnet/community/office-hours/index.rst Outdated
@pablozsc pablozsc merged commit 8ca6457 into main Apr 30, 2025
2 checks passed
@pablozsc pablozsc deleted the provenance-tags-interview branch April 30, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview Trigger a documentation preview for a PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants