Skip to content

Add saddle_points exercice#781

Open
resu-xuniL wants to merge 1 commit into
exercism:mainfrom
resu-xuniL:saddle-point
Open

Add saddle_points exercice#781
resu-xuniL wants to merge 1 commit into
exercism:mainfrom
resu-xuniL:saddle-point

Conversation

@resu-xuniL

Copy link
Copy Markdown
Contributor

Add saddle_points exercice to track.
I change difficulty from 1 to 4.

Hope everything is correct.


Reviewer Resources:

Track Policies

@resu-xuniL

Copy link
Copy Markdown
Contributor Author

what means 'foregone' ?

@glennj

glennj commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Before I look at this, please submit one PR at a time please. Let us reviewers deal with one at a time.

@glennj

glennj commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

what means 'foregone' ?

The maintainers of a track decide not to implement an exercise, then it goes on the "foregone" list to declare "we forego putting this one on the track". It might be too OO-specific for bash, or relies entirely on floats, or just something that really doesn't make sense to implement in shell.

@resu-xuniL

Copy link
Copy Markdown
Contributor Author

what means 'foregone' ?

The maintainers of a track decide not to implement an exercise, then it goes on the "foregone" list to declare "we forego putting this one on the track". It might be too OO-specific for bash, or relies entirely on floats, or just something that really doesn't make sense to implement in shell.

Oh I see!

@resu-xuniL

Copy link
Copy Markdown
Contributor Author

Before I look at this, please submit one PR at a time please. Let us reviewers deal with one at a time.

Oh! yes! sorry then

@glennj

glennj commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

So the failing check is directly due to this:

https://github.com/exercism/bash/actions/runs/27501739390/job/81285744611?pr=781#step:4:9

@resu-xuniL

Copy link
Copy Markdown
Contributor Author

So the failing check is directly due to this:

https://github.com/exercism/bash/actions/runs/27501739390/job/81285744611?pr=781#step:4:9

yes indeed.
Shall I close/remove this PR?

@glennj

glennj commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Please see issue #665 and PR #667 for the discussion

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