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

feat: allow access to remote state #158

Merged
merged 1 commit into from
Mar 23, 2024
Merged

Conversation

nadilas
Copy link
Member

@nadilas nadilas commented Mar 23, 2024

TL;DR

This pull request introduces changes and enhancements in the Repository class in 'repository.ts' and corresponding tests in 'commit.test.ts'. It also includes updates to the interface description in 'commit.ts'.

What changed?

The Repository class now has a new member 'remoteRefs' and a respective accessor function 'remote'. The member keeps track of the state of the remote during initialization. The new test checks for the status and commit operations on a Repository instance.

In 'commit.ts', block comments format has been introduced in place of line comments for commit hash details.

How to test?

Reviewers can run the included test in 'commit.test.ts' to verify the new changes functionality. Check updated commenting style in 'commit.ts'.

Why make this change?

The changes enhance the functionality of the Repository class and improve code readability in 'commit.ts' with proper block comments.


Copy link

vercel bot commented Mar 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
ogre ⬜️ Ignored (Inspect) Visit Preview Mar 23, 2024 0:24am

Copy link
Member Author

nadilas commented Mar 23, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nadilas and the rest of your teammates on Graphite Graphite

Copy link
Member Author

nadilas commented Mar 23, 2024

Merge activity

  • Mar 23, 1:23 PM GMT+1: @nadilas started a stack merge that includes this pull request via Graphite.
  • Mar 23, 1:25 PM GMT+1: Graphite rebased this pull request as part of a merge.
  • Mar 23, 1:27 PM GMT+1: @nadilas merged this pull request with Graphite.

@nadilas nadilas changed the base branch from fix/undefined-patch to main March 23, 2024 12:23
@nadilas nadilas force-pushed the feature/access-remote-state branch from cc74438 to e2d03f2 Compare March 23, 2024 12:24
@nadilas nadilas merged commit 96d0cf6 into main Mar 23, 2024
5 of 8 checks passed
@nadilas nadilas deleted the feature/access-remote-state branch March 23, 2024 12:27
Copy link

@github-actions github-actions bot added the released This issue/pull request has been released. label Mar 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant