| route-pattern | /_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format) |
| route-controller | voltron_pull_requests_fragments |
| route-action | pull_request_layout |
| fetch-nonce | v2:59abdcf9-6c2a-e713-b0a1-5d3c8717c1e1 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | AD5C:36DEBA:233D54F:2D9677A:697FD6FE |
| html-safe-nonce | 0a1fcdf546bb5653e42a57d01f2a375ffbd9544a1617cbaa821a71e61f49f7e2 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBRDVDOjM2REVCQToyMzNENTRGOjJEOTY3N0E6Njk3RkQ2RkUiLCJ2aXNpdG9yX2lkIjoiOTAzOTIxMjczNTg0OTU1MTYxNCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 7ed822291107e3ae137dd00a74d4bb75a8259b604690924c378a5e4893122a56 |
| hovercard-subject-tag | pull_request:520728422 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///voltron/pull_requests_fragments/pull_request_layout |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/osrf/ros2_java/53/pull_request_layout |
| twitter:image | https://opengraph.githubassets.com/532dc3f2670855ad82d88f0adfe0d0472ce1cee6ed666da6db1ae1390319a2b6/osrf/ros2_java/pull/53 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/532dc3f2670855ad82d88f0adfe0d0472ce1cee6ed666da6db1ae1390319a2b6/osrf/ros2_java/pull/53 |
| og:image:alt | Depends on #52
This is a partial implementation; trying to keep the diff smaller for review.
I'll follow-up with more PRs competing the implementation. |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | jacobperron |
| hostname | github.com |
| expected-hostname | github.com |
| None | 60279d4097367e16897439d16d6bbe4180663db828c666eeed2656988ffe59f6 |
| turbo-cache-control | no-preview |
| go-import | github.com/osrf/ros2_java git https://github.com/osrf/ros2_java.git |
| octolytics-dimension-user_id | 3999730 |
| octolytics-dimension-user_login | osrf |
| octolytics-dimension-repository_id | 278167259 |
| octolytics-dimension-repository_nwo | osrf/ros2_java |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | true |
| octolytics-dimension-repository_parent_id | 52850323 |
| octolytics-dimension-repository_parent_nwo | ros2-java/ros2_java |
| octolytics-dimension-repository_network_root_id | 52850323 |
| octolytics-dimension-repository_network_root_nwo | ros2-java/ros2_java |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 7c85641c598ad130c74f7bcc27f58575cac69551 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#start-of-content |
|
| https://patch-diff.githubusercontent.com/ |
|
Sign in
| https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fosrf%2Fros2_java%2Fpull%2F53 |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub SparkBuild and deploy intelligent apps | https://github.com/features/spark |
| GitHub ModelsManage and compare prompts | https://github.com/features/models |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| GitHub SponsorsFund open source developers | https://github.com/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/accelerator |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fosrf%2Fros2_java%2Fpull%2F53 |
|
Sign up
| https://patch-diff.githubusercontent.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=osrf%2Fros2_java |
| Reload | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| Reload | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| Reload | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
|
osrf
| https://patch-diff.githubusercontent.com/osrf |
| ros2_java | https://patch-diff.githubusercontent.com/osrf/ros2_java |
| ros2-java/ros2_java | https://patch-diff.githubusercontent.com/ros2-java/ros2_java |
|
Notifications
| https://patch-diff.githubusercontent.com/login?return_to=%2Fosrf%2Fros2_java |
|
Fork
9
| https://patch-diff.githubusercontent.com/login?return_to=%2Fosrf%2Fros2_java |
|
Star
4
| https://patch-diff.githubusercontent.com/login?return_to=%2Fosrf%2Fros2_java |
|
Code
| https://patch-diff.githubusercontent.com/osrf/ros2_java |
|
Pull requests
2
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pulls |
|
Actions
| https://patch-diff.githubusercontent.com/osrf/ros2_java/actions |
|
Projects
0
| https://patch-diff.githubusercontent.com/osrf/ros2_java/projects |
|
Security
0
| https://patch-diff.githubusercontent.com/osrf/ros2_java/security |
|
Insights
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pulse |
|
Code
| https://patch-diff.githubusercontent.com/osrf/ros2_java |
|
Pull requests
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pulls |
|
Actions
| https://patch-diff.githubusercontent.com/osrf/ros2_java/actions |
|
Projects
| https://patch-diff.githubusercontent.com/osrf/ros2_java/projects |
|
Security
| https://patch-diff.githubusercontent.com/osrf/ros2_java/security |
|
Insights
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pulse |
| Sign up for GitHub
| https://patch-diff.githubusercontent.com/signup?return_to=%2Fosrf%2Fros2_java%2Fissues%2Fnew%2Fchoose |
| terms of service | https://docs.github.com/terms |
| privacy statement | https://docs.github.com/privacy |
| Sign in | https://patch-diff.githubusercontent.com/login?return_to=%2Fosrf%2Fros2_java%2Fissues%2Fnew%2Fchoose |
| Jump to bottom | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#issue-comment-box |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| galactic-devel | https://patch-diff.githubusercontent.com/osrf/ros2_java/tree/galactic-devel |
| jacob/action_server | https://patch-diff.githubusercontent.com/osrf/ros2_java/tree/jacob/action_server |
|
Add action server with logic for accepting and canceling goals
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#top |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| galactic-devel | https://patch-diff.githubusercontent.com/osrf/ros2_java/tree/galactic-devel |
| jacob/action_server | https://patch-diff.githubusercontent.com/osrf/ros2_java/tree/jacob/action_server |
|
Conversation
74
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
|
Commits
32
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits |
|
Checks
0
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/checks |
|
Files changed
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| https://github.co/hiddenchars |
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/{{ revealButtonHref }} |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| Nov 13, 2020 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#issue-742643972 |
| #52 | https://github.com/osrf/ros2_java/pull/52 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
| November 13, 2020 17:39 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-3994571932 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| Nov 13, 2020 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-3994571975 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
|
Nov 13, 2020
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#ref-pullrequest-742641108 |
|
Add action interfaces for goal request and goal response
#52
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/52 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| November 16, 2020 18:35 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-4001761577 |
| https://patch-diff.githubusercontent.com/ivanpauno |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
|
Nov 16, 2020
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-531600837 |
|
View reviewed changes
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-531600837 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/include/org_ros2_rcljava_action_ActionServerImpl.h | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-8a8b3f9bc2f80d15b5089636b2b6d71403e1994242dc9a2e24dfe84f373707c5 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/cpp/org_ros2_rcljava_action_ActionServerImpl.cpp | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-08a75797b9f07509c3059f4b5af0b477d392f7a67d30c7412e5075ccb55312d2 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/cpp/org_ros2_rcljava_action_ActionServerImpl.cpp | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-08a75797b9f07509c3059f4b5af0b477d392f7a67d30c7412e5075ccb55312d2 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/cpp/convert.hpp | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-85b6959180d94d91c9addce72bca70861a8b5f5f92ba0e2444dd311becaf6619 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/cpp/org_ros2_rcljava_action_ActionServerImpl_GoalHandleImpl.cpp | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-c5498c58ce4df841a5ce978ead62367fde2e3dcb186e051dab139ac479275e7a |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| https://patch-diff.githubusercontent.com/ivanpauno |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
|
Jan 5, 2021
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-561146990 |
|
View reviewed changes
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-561146990 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/CMakeLists.txt | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-4f48a02bfcde36562ca555f728634ccbe6a7fe3d9569c2845c661b7ec88ebb95 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/CMakeLists.txt | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-4f48a02bfcde36562ca555f728634ccbe6a7fe3d9569c2845c661b7ec88ebb95 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/CMakeLists.txt | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-4f48a02bfcde36562ca555f728634ccbe6a7fe3d9569c2845c661b7ec88ebb95 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/action/ActionServerGoalHandle.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-b72e71a5986085296d5cd68f19e7f1ec03f92a4814cf3d437529bd21d40ea3df |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/action/ActionServerGoalHandle.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-b72e71a5986085296d5cd68f19e7f1ec03f92a4814cf3d437529bd21d40ea3df |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/action/ActionServerImpl.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-d5e31f2870f8429406fbba4938fc4b68010d87fe52fc848366c921bb97acdc93 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/action/CancelCallback.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-a3b96ca1a1aacae8f566f621bbffc05359e149e86b8e9c287851788d2d6fb57b |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/action/CancelCallback.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-a3b96ca1a1aacae8f566f621bbffc05359e149e86b8e9c287851788d2d6fb57b |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/action/GoalCallback.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-6153a641deb671a7a88397615b3deeab016f446f47336bbf1f6bf6257304c2c2 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/action/ActionServerImpl.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-d5e31f2870f8429406fbba4938fc4b68010d87fe52fc848366c921bb97acdc93 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| force-pushed | https://patch-diff.githubusercontent.com/osrf/ros2_java/compare/0230bb20cb988c85feeb6c1978d71f87d24bb76b..21b194a1a5c5b016c2356b086dc1ea8621483ce4 |
| 0230bb2 | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/0230bb20cb988c85feeb6c1978d71f87d24bb76b |
| 21b194a | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/21b194a1a5c5b016c2356b086dc1ea8621483ce4 |
|
Compare
| https://patch-diff.githubusercontent.com/osrf/ros2_java/compare/0230bb20cb988c85feeb6c1978d71f87d24bb76b..21b194a1a5c5b016c2356b086dc1ea8621483ce4 |
| January 13, 2021 20:16 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-4203043052 |
| https://patch-diff.githubusercontent.com/ivanpauno |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
|
Jan 18, 2021
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-570500165 |
|
View reviewed changes
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-570500165 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/cpp/convert.hpp | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-85b6959180d94d91c9addce72bca70861a8b5f5f92ba0e2444dd311becaf6619 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/cpp/convert.hpp | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-85b6959180d94d91c9addce72bca70861a8b5f5f92ba0e2444dd311becaf6619 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/executors/BaseExecutor.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-f575b3e4d253d9d5c03ac4fe0c389a3f9ad4d2f67203cf9c85f7fa4577c4caf6 |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| rcljava/src/main/java/org/ros2/rcljava/node/Node.java | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files#diff-9f5bf036be2b893d8b72c636659af84f6439f3d0d372af4816022e7bab14a40a |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| January 20, 2021 11:06 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#commits-pushed-4a5d229 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add interfaces for action goal, result, and feedback | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/4a5d229fecdb74d87a18322dc16ae5a20cdfef4a |
| 4a5d229 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/4a5d229fecdb74d87a18322dc16ae5a20cdfef4a |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add new definitions for action goal response and request | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/768b6d470200dad2aa5946a465d941b56dc091d0 |
| 768b6d4 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/768b6d470200dad2aa5946a465d941b56dc091d0 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add getter for UUID to SendGoalRequest | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/bdecfa7aef77d5013c6b42f91bd354b7d2e50396 |
| bdecfa7 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/bdecfa7aef77d5013c6b42f91bd354b7d2e50396 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add getStamp method to GoalResponseDefinition | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/1b65d7fd91a95cd7e112c33d1f4ea8f2312a6bb6 |
| 1b65d7f | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/1b65d7fd91a95cd7e112c33d1f4ea8f2312a6bb6 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Partially revert "Add interfaces for action goal, result, and feedback" | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/b725144feda52eadff2e1d8b46d995bab12d0298 |
| b725144 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/b725144feda52eadff2e1d8b46d995bab12d0298 |
| dd04614 | https://github.com/osrf/ros2_java/commit/dd046147d38c1aa844376149f62c4f3aaec04d96 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Parameterize goal request and response interfaces on action type | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f7d7f557acc63f05804a33daaa212e13090bb208 |
| f7d7f55 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f7d7f557acc63f05804a33daaa212e13090bb208 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| force-pushed | https://patch-diff.githubusercontent.com/osrf/ros2_java/compare/492496557172241dd38db10bb36fbf85e1c2846b..f7d7f557acc63f05804a33daaa212e13090bb208 |
| 4924965 | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/492496557172241dd38db10bb36fbf85e1c2846b |
| f7d7f55 | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/f7d7f557acc63f05804a33daaa212e13090bb208 |
|
Compare
| https://patch-diff.githubusercontent.com/osrf/ros2_java/compare/492496557172241dd38db10bb36fbf85e1c2846b..f7d7f557acc63f05804a33daaa212e13090bb208 |
| January 20, 2021 19:06 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-4229504294 |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| January 20, 2021 11:55 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#commits-pushed-7a6a47f |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Fix getGoalUuid implementation | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/7a6a47f645c5afedbb39b760c0ec685dfcb0b5ef |
| 7a6a47f | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/7a6a47f645c5afedbb39b760c0ec685dfcb0b5ef |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| List -> byte[] | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/63200b3af16a1afd3d01f69e868b8524e4db5851 |
| 63200b3 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/63200b3af16a1afd3d01f69e868b8524e4db5851 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add ActionServer skeleton | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/cd347890b9809d4a100aa30696db0b8edac2102d |
| cd34789 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/cd347890b9809d4a100aa30696db0b8edac2102d |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add ActionServer creation logic and unit tests | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f988e401674b723ad74dbb92b8a25f06f5ece5e4 |
| f988e40 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f988e401674b723ad74dbb92b8a25f06f5ece5e4 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add action server logic for accepting and canceling goals | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/676fcae11dee0f01eae1c897c586b7769040b4bd |
| 676fcae | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/676fcae11dee0f01eae1c897c586b7769040b4bd |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| cleanup | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/0494903042df03e7c982da8e35a3cbb5579bbfbd |
| 0494903 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/0494903042df03e7c982da8e35a3cbb5579bbfbd |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| more cleanup | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/5a50ddcf484c108a4d77ba0f9352ca6a71246623 |
| 5a50ddc | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/5a50ddcf484c108a4d77ba0f9352ca6a71246623 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Minor refactor: move response enums into callback interfaces | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/af943df368093c92d81e08e29a585c4a0460d10b |
| af943df | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/af943df368093c92d81e08e29a585c4a0460d10b |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Remove TODO | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/7929ece50dbc06845cbff4b1521e8cb1755aab2a |
| 7929ece | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/7929ece50dbc06845cbff4b1521e8cb1755aab2a |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Fix accident | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/98fd96f67ac5a8b997a8badba792bd42b4bd004a |
| 98fd96f | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/98fd96f67ac5a8b997a8badba792bd42b4bd004a |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Alphabetize | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/1a3403866121c6475e0202fbc7a2d1e91c259a96 |
| 1a34038 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/1a3403866121c6475e0202fbc7a2d1e91c259a96 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Refactor: replace 'getNumberOf*()' JNI functions with single 'getNumb… | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/08d180a911c93a2135e225ce27936edd6ed56008 |
| 08d180a | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/08d180a911c93a2135e225ce27936edd6ed56008 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Avoid string copy | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/ac784b3d9c9825ffdf30497bbacb58b5ae28d2ad |
| ac784b3 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/ac784b3d9c9825ffdf30497bbacb58b5ae28d2ad |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Remove extern C | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/d61317534816c92a051a97a9638bfc2d30fc9550 |
| d613175 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/d61317534816c92a051a97a9638bfc2d30fc9550 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Remove unnecessary synchronization | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/ed684b12201ed579259d8356cad93e6e25634a60 |
| ed684b1 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/ed684b12201ed579259d8356cad93e6e25634a60 |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| January 20, 2021 11:56 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#commits-pushed-f2de762 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Refactor access to goalCallback | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f2de7627cebd7c19447be5c2f2d03afbb4db2c8b |
| f2de762 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f2de7627cebd7c19447be5c2f2d03afbb4db2c8b |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| typesafe request and response definitions | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/11b107e2cd0d5258efeba326da82789544da514b |
| 11b107e | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/11b107e2cd0d5258efeba326da82789544da514b |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Be more specific about template type for GoalCallback | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/6203cead13af7973abbfdab409d6a6c25c809058 |
| 6203cea | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/6203cead13af7973abbfdab409d6a6c25c809058 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Minor refactor | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/5e4693ad1472d96b3c1eed4de4d31ba35b11dc3d |
| 5e4693a | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/5e4693ad1472d96b3c1eed4de4d31ba35b11dc3d |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Make GoalHandleImpl inner class instead of static | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/9c93a03be4f5ff0a1ac024cfd05645f788b243c3 |
| 9c93a03 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/9c93a03be4f5ff0a1ac024cfd05645f788b243c3 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Remove synchronized from getHandle | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f6dbc953cdb793ad4a9732aa772b6fb893503c0c |
| f6dbc95 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f6dbc953cdb793ad4a9732aa772b6fb893503c0c |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add TODO for Waitable interface | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/c652771557d7fcf3809f64b243898af79f89d51c |
| c652771 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/c652771557d7fcf3809f64b243898af79f89d51c |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Fix style | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/59b2d2464ea4b1cba309599a6410cde808cf1920 |
| 59b2d24 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/59b2d2464ea4b1cba309599a6410cde808cf1920 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Minor refactor | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/b4b79229437ebb5e20e74fb3cfe9df1612435565 |
| b4b7922 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/b4b79229437ebb5e20e74fb3cfe9df1612435565 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add docs for createActionServer | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/6014fad407d9f11ab7c6eada9c1e142da3cfa26d |
| 6014fad | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/6014fad407d9f11ab7c6eada9c1e142da3cfa26d |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Switch from List to array | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
| f94150b | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/commits/f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| force-pushed | https://patch-diff.githubusercontent.com/osrf/ros2_java/compare/6fe23e6423821671b29e26cfd29fc2e527fcd7d2..f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
| 6fe23e6 | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/6fe23e6423821671b29e26cfd29fc2e527fcd7d2 |
| f94150b | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
|
Compare
| https://patch-diff.githubusercontent.com/osrf/ros2_java/compare/6fe23e6423821671b29e26cfd29fc2e527fcd7d2..f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
| January 20, 2021 20:42 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-4229869439 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| Jan 20, 2021 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#issuecomment-763920968 |
| @ivanpauno | https://github.com/ivanpauno |
| f94150b | https://github.com/osrf/ros2_java/commit/f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| January 20, 2021 21:02 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-4229941451 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
| January 20, 2021 21:49 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-4230139452 |
| https://patch-diff.githubusercontent.com/ivanpauno |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
|
Jan 21, 2021
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-573306088 |
|
View reviewed changes
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files/f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#pullrequestreview-573306088 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| af0ba27 | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/af0ba277e26a5ce7c9a2ba9464616f473d4931a8 |
| Jan 21, 2021 | https://github.com/osrf/ros2_java/pull/53#event-4234916752 |
| https://patch-diff.githubusercontent.com/jacobperron |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
| January 21, 2021 18:08 | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#event-4234917035 |
| ivanpauno | https://patch-diff.githubusercontent.com/ivanpauno |
|
May 17, 2021
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#ref-commit-187e661 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
|
| https://patch-diff.githubusercontent.com/ivanpauno |
| Add action server with logic for accepting and canceling goals ( | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/187e6614632f98d8395d1d7978d7a3deca4fd8ee |
| #53 | https://github.com/osrf/ros2_java/pull/53 |
| ) | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/187e6614632f98d8395d1d7978d7a3deca4fd8ee |
| 187e661 | https://patch-diff.githubusercontent.com/osrf/ros2_java/commit/187e6614632f98d8395d1d7978d7a3deca4fd8ee |
| dd04614 | https://github.com/osrf/ros2_java/commit/dd046147d38c1aa844376149f62c4f3aaec04d96 |
| jacobperron | https://patch-diff.githubusercontent.com/jacobperron |
|
May 17, 2022
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53#ref-commit-d2ea45d |
|
| https://patch-diff.githubusercontent.com/jacobperron |
| Add action server with logic for accepting and canceling goals ( | https://patch-diff.githubusercontent.com/ros2-java/ros2_java/commit/d2ea45d985dae4e3bb68559284a5191a24f2be37 |
| osrf#53 | https://github.com/osrf/ros2_java/pull/53 |
| ) | https://patch-diff.githubusercontent.com/ros2-java/ros2_java/commit/d2ea45d985dae4e3bb68559284a5191a24f2be37 |
| d2ea45d | https://patch-diff.githubusercontent.com/ros2-java/ros2_java/commit/d2ea45d985dae4e3bb68559284a5191a24f2be37 |
| dd04614 | https://github.com/ros2-java/ros2_java/commit/dd046147d38c1aa844376149f62c4f3aaec04d96 |
| Sign up for free | https://patch-diff.githubusercontent.com/join?source=comment-repo |
| Sign in to comment | https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fosrf%2Fros2_java%2Fpull%2F53 |
|
| https://patch-diff.githubusercontent.com/ivanpauno |
|
ivanpauno
| https://patch-diff.githubusercontent.com/ivanpauno |
|
| https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53/files/f94150b1a8a9bbf87e299471f3c632acca4e9b6e |
|
| https://patch-diff.githubusercontent.com/jacobperron |
|
jacobperron
| https://patch-diff.githubusercontent.com/jacobperron |
| Please reload this page | https://patch-diff.githubusercontent.com/osrf/ros2_java/pull/53 |
|
| https://patch-diff.githubusercontent.com/jacobperron |
|
| https://patch-diff.githubusercontent.com/ivanpauno |
|
| https://patch-diff.githubusercontent.com/apps/copilot-swe-agent |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |