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

Krishna mk patch 2 #52

Closed

Conversation

krishnamk00
Copy link
Contributor

No description provided.

@cfdreddbot
Copy link

Hey krishna-mk!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you've already signed the CLA.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/96957948.

@krishnamk00
Copy link
Contributor Author

Giving the correct pointers/links instead of text strings.

@emalm
Copy link
Member

emalm commented Jun 29, 2015

Hi, @Krishna-Mk,

With your changes, I don't think the 'Follow the READMEs in their respective repositories:' text that remains in that section makes sense any more. Could you please remove it as well?

Also, I'm surprised to see 4 commits already on this PR to change a few lines of text. Maybe with that additional change you could squash them down to a single commit and resubmit the PR?

Thanks,
Eric

jvshahid pushed a commit that referenced this pull request Oct 19, 2017
[finishes #150821436]

closes #353

Submodule src/code.cloudfoundry.org/dockerapplifecycle 9807a9c0..090d29da:
  > Replace the manifest with a more recent manifest
Submodule src/github.com/docker/distribution ec87e9b69...916276044:
  > Merge pull request #1271 from aaronlehmann/remove-tarsum
  > Merge pull request #1264 from aaronlehmann/hash-errors
  > Merge pull request #1258 from thaJeztah/fix-docs-headers
  > Merge pull request #1267 from gabriellhrn/typo
  > Merge pull request #1245 from carsonoid/carsonoid-nginx-doc-patch-2
  > Merge pull request #1253 from RichardScothern/go-1.5.2
  > Merge pull request #1250 from RichardScothern/AUTHORS
  > Merge pull request #1187 from stevvooe/check-storage-drivers-list-path-not-found
  > Merge pull request #1246 from stevvooe/decrease-driver-testsuite-allocations
  > Merge pull request #1237 from thaJeztah/update-maintainers
  > Merge pull request #1238 from noxiouz/fix_comment
  > Merge pull request #1241 from pdevine/catalog-speedup
  > Merge pull request #1243 from RichardScothern/client-header-message
  > Merge pull request #1157 from lebauce/swift-bulk-delete-fixes
  > Merge pull request #1239 from stevvooe/refactor-reference-grammar
  > Merge pull request #1234 from SvenDowideit/linkchecker-fixes
  > Merge pull request #1233 from kendru/patch-1
  > Merge pull request #1231 from tonistiigi/digest-length-check
  > Merge pull request #1211 from tt/remove-name-verification
  > Merge pull request #1232 from pdevine/catalog-clarification
  > Merge pull request #1227 from stevvooe/walk-sorted
  > Merge pull request #1226 from aaronlehmann/avoid-stat-roundtrips
  > Merge pull request #1225 from SvenDowideit/DOCS-227
  > Merge pull request #1213 from SvenDowideit/move-to-docs-base-latest
  > Merge pull request #1208 from farmerworking/fix-oss-doc-error
  > Merge pull request #1204 from MHBauer/gh1197
  > Merge pull request #1207 from Djelibeybi/master
  > Merge pull request #1198 from jonboulle/master
  > Merge pull request #1193 from SvenDowideit/docs-validation-changes
  > Merge pull request #1166 from treed/master
  > Merge pull request #910 from RichardScothern/go-1.5
  > Merge pull request #1125 from rustyconover/fix-url-path-escaping
  > Merge pull request #1141 from lebauce/swift-no-missing-segment
  > Merge pull request #1172 from aaronlehmann/swift-empty-delete-request
  > Merge pull request #1156 from RichardScothern/manifest-verification
  > Merge pull request #1161 from stevvooe/add-status-badges
  > Merge pull request #1150 from RichardScothern/carry-1070
  > Merge pull request #1114 from lebauce/swift-temp-url
  > Merge pull request #1152 from RichardScothern/credit2
  > Merge pull request #1151 from RichardScothern/credit
  > Merge pull request #1149 from stevvooe/close-filereader
  > Merge pull request #1148 from aaronlehmann/withname-check-length
  > Merge pull request #1122 from RichardScothern/scheduler-coalesce
  > Merge pull request #1139 from aaronlehmann/reference-withtag-fix
  > Merge pull request #756 from aibaars/storage-gcs
  > Merge pull request #1096 from aaronlehmann/simplify-proxy-scheduler
  > Merge pull request #1136 from RichardScothern/update-roadmap
  > Merge pull request #1047 from mattmoor/expires_in_proposal
  > Merge pull request #1099 from RichardScothern/go-version
  > Merge pull request #1121 from thaJeztah/fix-doc-emoji
  > Merge pull request #1095 from mrwacky42/api-version-annotated
  > Merge pull request #1118 from SvenDowideit/stop-linking-when-rendering-to-html
  > Merge pull request #1098 from denverdino/master
  > Merge pull request #1115 from tianon/fix-auth-servehttp-comment
  > Merge pull request #1111 from aaronlehmann/golint-comments
  > Merge pull request #1110 from RichardScothern/env-var-clarification
  > Merge pull request #1006 from RichardScothern/proxy-serialize
  > Merge pull request #1103 from moxiegirl/hugo-15-retooling
  > Merge pull request #827 from aaronlehmann/read-only-mode-2
  > Merge pull request #1053 from mrwacky42/patch-1
  > Merge pull request #1094 from aaronlehmann/correct-payload-serialization-order
  > Merge pull request #1093 from aaronlehmann/reference-improvements
  > Merge pull request #1035 from aaronlehmann/host-parameter-alternate
  > Merge pull request #1033 from jlhawn/api_spec_update_auth_errors
  > Merge pull request #963 from dmcgowan/reference-update
  > Merge pull request #1083 from BrianBland/fixSwiftDocs
  > Merge pull request #1079 from mattmoor/allow-hostname-parts
  > Merge pull request #1072 from compnski/patch-1
  > Merge pull request #1076 from harche/patch-2
  > Merge pull request #1048 from docker/docs-secure-default
  > Merge pull request #950 from jackpgriffin/master
  > Merge pull request #1005 from xiekeyang/spec
  > Merge pull request #988 from tt/fix-incorrect-default
  > Merge pull request #1045 from tt/remove-duplicate-dependency-definition
  > Merge pull request #1036 from dmcgowan/digest-set-update
  > Merge pull request #1024 from aaronlehmann/avoid-testing-import
  > Merge pull request #1042 from tt/upgrade-amazon-library
  > Merge pull request #1001 from RichardScothern/fix-atrocity
  > Merge pull request #991 from mattmoor/expires_in_proposal
  > Merge pull request #999 from nakedible-p/patch-1
  > Merge pull request #1014 from shyr/patch-3
  > Merge pull request #997 from xiekeyang/typo
  > Merge pull request #981 from LouisKottmann/master
  > Merge pull request #973 from jlhawn/update_token_spec
  > Merge pull request #974 from stevvooe/context-cleanup
  > Merge pull request #977 from aaronlehmann/use-distribution-context
  > Merge pull request #909 from aaronlehmann/importable-registry
  > Merge pull request #888 from aaronlehmann/config-env-vars
  > Merge pull request #964 from aaronlehmann/pin-dind-dep
  > Merge pull request #946 from humble00/master
  > Merge pull request #949 from docker/amylindburg-patch-1
  > Merge pull request #941 from liubin/master
  > Merge pull request #931 from dmp42/5.all-ur-proxy-are-belong-to-us
  < Correctly sanitize location url preserving parameters
  > Merge pull request #934 from stevvooe/sanitize-url-correctly
  > Merge pull request #929 from RichardScothern/docs
  > Merge pull request #932 from avinson/master
  > Merge pull request #918 from aaronlehmann/json-spec
  > Merge pull request #897 from dmp42/5.all-ur-proxy-are-belong-to-us
  > Merge pull request #912 from stevvooe/manifest-refactoring-schema1
  > Merge pull request #907 from stevvooe/digest-tool
  > Merge pull request #876 from dmcgowan/integration-test-cleanup
  > Merge pull request #901 from aaronlehmann/configurable-health-checks
  > Merge pull request #893 from aaronlehmann/functional-opts
  > Merge pull request #896 from aaronlehmann/closenotifier-warnings
  > Merge pull request #894 from aaronlehmann/fix-tests
  > Merge pull request #874 from stevvooe/load-balancer-endpoint
  > Merge pull request #846 from aaronlehmann/http-header-configuration
  > Merge pull request #883 from tg123/master
  > Merge pull request #889 from stevvooe/remove-pathmapper
  > Merge pull request #881 from samsong8610/update-example-config-to-use-blobdescriptor
  > Merge pull request #869 from mssola/delete-documentation
  > Merge pull request #864 from stevvooe/use-correct-manifest-link
  > Merge pull request #865 from dmp42/4.docs-2.1
  > Merge pull request #862 from stevvooe/dont-comment-on-closed-issues
  > Merge pull request #856 from dmp42/4.docs-2.1
  > Merge pull request #847 from aaronlehmann/harmonize-config-paths
  > Merge pull request #848 from RichardScothern/error-type
  > Merge pull request #853 from dmp42/4.docs-2.1
  > Merge pull request #852 from ahmetalpbalkan/docs/typo
  > Merge pull request #831 from stevvooe/add-driver-health-check
  > Merge pull request #851 from ahmetalpbalkan/docs/redirect
  > Merge pull request #821 from aaronlehmann/docker-integration-fixes-3
  > Merge pull request #845 from gierschv/add-rados-root-omap
  > Merge pull request #837 from dmp42/4.docs-2.1
  > Merge pull request #839 from n1tr0g/master
  > Merge pull request #836 from stevvooe/fix-mailmap-scothern
  > Merge pull request #835 from stevvooe/update-authors
  > Merge pull request #833 from stevvooe/remove-dist-tool
  > Merge pull request #834 from stevvooe/rev-default-version
  > Merge pull request #830 from aaronlehmann/no-closenotifier-panic
  > Merge pull request #818 from dmp42/3.fix-build-docs
  > Merge pull request #829 from aaronlehmann/aliyun-oss-doc-styling
  > Merge pull request #779 from RichardScothern/pull-through-cache
  > Merge pull request #816 from mbentley/add-migrator
  > Merge pull request #805 from RichardScothern/811
  > Merge pull request #817 from aaronlehmann/docker-integration-fixes-2
  > Merge pull request #514 from denverdino/master
  > Merge pull request #815 from aaronlehmann/docker-integration-fixes
  > Merge pull request #814 from aaronlehmann/test-response-writer
  > Merge pull request #783 from aaronlehmann/multiple-version-integration-testing
  > Merge pull request #763 from aaronlehmann/close-notifier
  > Merge pull request #803 from duglin/FixBuild
  > Merge pull request #811 from aaronlehmann/fix-vet
  > Merge pull request #812 from stevvooe/update-s3-dependency
  > Merge pull request #801 from alexwlchan/master
Submodule src/github.com/docker/libtrust fa567046d..aabc10ec2:
  > Merge pull request #56 from thaJeztah/add-deprecation-warning
  > Merge pull request #54 from stweil/master
  > Merge pull request #52 from vbatts/vbatts-test_base64url
  > Merge pull request #49 from stevvooe/signature-assembly

Signed-off-by: John Shahid <jvshahid@gmail.com>
ameowlia added a commit that referenced this pull request Jan 13, 2023
Submodule src/code.cloudfoundry.org/buildpackapplifecycle 7a17f3d0f..2b9331e66:
  > Merge pull request #52 from dsabeti/future-proof-git-fixtures
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.

None yet

4 participants