Skip to content

Conversation

@jamesbhobbs
Copy link
Contributor

@jamesbhobbs jamesbhobbs commented Oct 20, 2025

Remove andyjakubowski from CODEOWNERS

Summary

Removes @andyjakubowski from the default code owners list for the vscode-deepnote repository. This user will no longer be automatically requested for code reviews on pull requests.

This change is part of a broader cleanup to remove this user from CODEOWNERS across multiple Deepnote repositories (jupyterlab-deepnote, deepnote, vscode-deepnote, and deepnote-internal).

Changes

Review Checklist


Link to Devin run: https://app.devin.ai/sessions/5a8f5921776546b797c99b41edeb78bb
Requested by: James Hobbs (james@deepnote.com) - @jamesbhobbs

Summary by CodeRabbit

  • Chores
    • Updated the default code owners configuration.

@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 20, 2025

📝 Walkthrough

Walkthrough

Updated the default CODEOWNERS entry in .github/CODEOWNERS by removing @andyjakubowski from the default owners list. The repository's default code reviewers are now @saltenasl, @jamesbhobbs, and @Artmann.

Possibly related PRs

Suggested reviewers

  • Artmann

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "Remove andyjakubowski from CODEOWNERS" directly and accurately describes the primary change in the PR. It's specific, concise, and contains no vague language or extraneous details. A reviewer scanning the repository history would immediately understand that this commit removes a user from the CODEOWNERS file, which matches exactly what the changeset does.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 70a2459 and 9509af1.

📒 Files selected for processing (1)
  • .github/CODEOWNERS (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Build & Test
  • GitHub Check: Build & Package Extension
🔇 Additional comments (1)
.github/CODEOWNERS (1)

2-2: Looks good.

The removal is clean and the CODEOWNERS syntax is correct.


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

@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71%. Comparing base (70a2459) to head (9509af1).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #94   +/-   ##
=====================================
  Coverage     71%     71%           
=====================================
  Files        523     523           
  Lines      39001   39001           
  Branches    4905    4905           
=====================================
  Hits       27834   27834           
  Misses      9542    9542           
  Partials    1625    1625           
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jamesbhobbs jamesbhobbs marked this pull request as ready for review October 20, 2025 18:36
@jamesbhobbs jamesbhobbs merged commit e6c8338 into main Oct 20, 2025
12 checks passed
@jamesbhobbs jamesbhobbs deleted the devin/1760985189-remove-andyjakubowski branch October 20, 2025 18:36
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.

2 participants