Skip to content

docs: fix career link - #122

Merged
jamesbhobbs merged 2 commits into
deepnote:mainfrom
zdenekornst:fix-career-link
Nov 4, 2025
Merged

docs: fix career link#122
jamesbhobbs merged 2 commits into
deepnote:mainfrom
zdenekornst:fix-career-link

Conversation

@zdenekornst

@zdenekornst zdenekornst commented Nov 4, 2025 •
edited by saltenasl
Loading

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Documentation
    • Updated the "join our team" link in README.

@zdenekornst
zdenekornst requested a review from a team as a code owner November 4, 2025 15:21
@coderabbitai

coderabbitai Bot commented Nov 4, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The README's Contributing section is updated, replacing two occurrences of the URL https://deepnote.com/careers with https://deepnote.com/join-us. This is a documentation-only change with no impact to code logic or exported declarations.

Possibly related PRs

Suggested reviewers

  • saltenasl
  • Artmann

Pre-merge checks

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title 'docs: fix career link' directly matches the changeset: two URLs in README.md's Contributing section updated from careers to join-us endpoint.
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 43eab34 and f101286.

📒 Files selected for processing (1)
  • README.md (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{md,yml,yaml}

📄 CodeRabbit inference engine (CLAUDE.md)

Use Prettier formatting for Markdown and YAML files

Files:

  • README.md
🔇 Additional comments (2)
README.md (2)

113-113: AI summary claims two occurrences, but only one change is shown.

The summary states "two occurrences" were updated, but only line 113 is marked as changed in the provided code. Verify whether a second career link should have been updated but wasn't.

113-113: URL update is correct.

The link now correctly points to https://deepnote.com/join-us instead of the deprecated careers URL. Markdown syntax is proper.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@saltenasl
saltenasl enabled auto-merge (squash) November 4, 2025 15:25
@jamesbhobbs

jamesbhobbs commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

@zdenekornst thank you!

@jamesbhobbs
jamesbhobbs disabled auto-merge November 4, 2025 18:55
@jamesbhobbs
jamesbhobbs merged commit 6350b1c into deepnote:main Nov 4, 2025
10 checks passed
@voyti voyti mentioned this pull request Sep 18, 2026
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jamesbhobbs jamesbhobbs jamesbhobbs approved these changes
@coderabbitai coderabbitai[bot] coderabbitai[bot] 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.

2 participants

@zdenekornst @jamesbhobbs