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

Supply missing right parenthesis #27

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Conversation

ffoulkes
Copy link
Contributor

@ffoulkes ffoulkes commented Apr 28, 2024

Two previous attempts to fix this have been blocked because the commits weren't signed.
Third time's a charm?

Signed-off-by: Derek G Foster <[email protected]>
@fruffy
Copy link

fruffy commented Apr 28, 2024

Who is currently maintaining target-syslibs? Any designated maintainers?

@ffoulkes
Copy link
Contributor Author

Who is currently maintaining target-syslibs? Any designated maintainers?

I don't know, but I'll ask around.

@fruffy
Copy link

fruffy commented Apr 29, 2024

Who is currently maintaining target-syslibs? Any designated maintainers?

I don't know, but I'll ask around.

Thanks! I am currently cleaning up the permission on all p4lang repositories. I have been also looking at syslibs and whether maintainers need to be removed or added.

If there is no one available to merge, I can also do it. Both pull requests look benign to me.

@saynb
Copy link
Collaborator

saynb commented Apr 30, 2024

Who is currently maintaining target-syslibs? Any designated maintainers?
Right now I am the only maintainer. But let me see if I can get some more people for these repos.

I am not sure why this PR is complaining about parenthesis

@saynb saynb merged commit e475d46 into p4lang:main Apr 30, 2024
2 checks passed
@fruffy
Copy link

fruffy commented Apr 30, 2024

Right now I am the only maintainer. But let me see if I can get some more people for these repos.

Please let me know. In the meantime I will remove everyone else.

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.

3 participants