profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/jparise/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Jon Parise jparise Pinterest Oakland, CA https://www.indelible.org/ Scientia Potentia Est

jparise/chrome-utm-stripper 632

Browser extension that strips Google Analytics (UTM) parameters, and various other click tracking tokens, from URL query strings

jparise/AFHTTPClientLogger 40

Configurable HTTP request logger for AFNetworking

jparise/apprankings 36

App Store Rankings Scraper

jparise/dotfiles 6

My public dotfiles

jparise/django-ink 3

Ink is a blogging application for Django.

jparise/AFHTTPRequestOperationLogger 2

AFNetworking Extension for HTTP Request Logging

jparise/dammit-winamp 2

DAMMIT WinAMP plugin that Soco and I wrote at Computer Science House in the fall of 1998

jparise/ActionSheetPicker 1

Quickly reproduce the dropdown UIPickerView / ActionSheet functionality from Safari on iPhone/ iOS / CocoaTouch.

jparise/AFNetworking 1

A delightful iOS networking library with NSOperations and block-based callbacks

pull request commentbazelbuild/bazel

Add {objc,objcpp}_compile actions to action sets

Alternatively, we could remove these default sets entirely because none of them are used here other than all_link_actions.

jparise

comment created time in 7 hours

PR opened bazelbuild/bazel

Add {objc,objcpp}_compile actions to action sets

The Objective-C compile actions weren't included in the 'all_compile_actions', 'preprocessor_compile_actions', and 'codegen_compile_actions' sets.

Adding them makes these action sets a bit more useful as demonstrated by the fact that we can now use them to set some default compiler flags below.

+8 -28

0 comment

1 changed file

pr created time in 8 hours

create barnchjparise/bazel

branch : objc_compile_actions

created branch time in 8 hours

fork jparise/bazel

a fast, scalable, multi-language and extensible build system

https://bazel.build

fork in 8 hours

issue commentPyCQA/pep8-naming

TypedDict classes should be exempt from N815

We actually now have code that attempts to traverse as much of the class hierarchy as is possible (given the AST's limitations). We use it to find Exception subclasses.

https://github.com/PyCQA/pep8-naming/blob/9adbbc4a6248db25eff2c7bc13b090b1d83ffd04/src/pep8ext_naming.py#L300-L316

A path to implementing this suggestion in this issue might be:

  • Generalize the code above
  • Introduce an ignore-class-names (or similar) configuration option which skips all (or a subset of?) checks for matching subclasses
  • Consider defaulting the list of ignored class names to ["TypedDict"]
gaborbernat

comment created time in 2 days

issue commentPyCQA/pep8-naming

Broken test with flake8>=3.9.1

I believe this was fixed in the last release. Can you confirm, @mweinelt?

mweinelt

comment created time in 2 days

startedgrailbio/bazel-toolchain

started time in 4 days

push eventjparise/.github

Jon Parise

commit sha c7898b63ed40966ca33c94a08fb39990f998856a

Remove PayPal link

view details

push time in 4 days

push eventpinterest/pymemcache

Tim Gates

commit sha b515ab6c99a4912fed389c011db0ae425ee3ab44

docs: Fix a few typos (#348)

view details

push time in 5 days

PR merged pinterest/pymemcache

Reviewers
docs: Fix a few typos

There are small typos in:

  • pymemcache/client/base.py
  • pymemcache/test/test_client.py
  • pymemcache/test/test_client_retry.py

Fixes:

  • Should read setting rather than settting.
  • Should read collection rather than collectino.
  • Should read serializer rather than seralizer.
  • Should read interaction rather than interacction.

Semi-automated pull request generated by https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

+8 -8

0 comment

3 changed files

timgates42

pr closed time in 5 days

PullRequestReviewEvent

pull request commentjparise/vim-graphql

Add rescript support

@amiralies Thanks! I'm not familiar with rescript so it will take be a little bit of time to come up to speed on this, but it looks like a simple-enough change.

Could you also update the README.md file and documentation (doc/graphql.txt) to mention this new language support?

amiralies

comment created time in 5 days

issue commentmicrosoft/pyright

Missing support for value customization with Enum using __new__

I've just run into this, too. I agree it's a rare case that can be avoided using different code patterns, but it does represent an officially supported way of using the enum module, so I think this may be worth revisiting for that reason alone.

I looked at Mypy, and it appears some similar issues have been raised there recently:

  • https://github.com/python/mypy/issues/10469
  • https://github.com/python/mypy/issues/10573
jhominal

comment created time in 6 days

created tagpinterest/PodToBUILD

tag4.0.0-599a84c

An easy way to integrate CocoaPods into Bazel

created time in 6 days

release pinterest/PodToBUILD

4.0.0-599a84c

released time in 6 days

push eventjparise/PodToBUILD

Jon Parise

commit sha 599a84c5d25bd55af32df685a114498b4a241ebe

Stop setting -DNS_BLOCK_ASSERTIONS=1 for :release (#186) As of Bazel 3.5.0, this is already set for the :opt profile on all Apple platforms (which is what :release uses): https://github.com/bazelbuild/bazel/commit/174ed30685ef3ed30e50e01f547c2fb7a003c12e

view details

push time in 6 days

PullRequestReviewEvent

delete branch jparise/PodToBUILD

delete branch : NS_BLOCK_ASSERTIONS

delete time in 7 days

push eventpinterest/PodToBUILD

Jon Parise

commit sha 599a84c5d25bd55af32df685a114498b4a241ebe

Stop setting -DNS_BLOCK_ASSERTIONS=1 for :release (#186) As of Bazel 3.5.0, this is already set for the :opt profile on all Apple platforms (which is what :release uses): https://github.com/bazelbuild/bazel/commit/174ed30685ef3ed30e50e01f547c2fb7a003c12e

view details

push time in 7 days

PR merged pinterest/PodToBUILD

Stop setting -DNS_BLOCK_ASSERTIONS=1 for :release

As of Bazel 3.5.0, this is already set for the :opt profile on all Apple platforms (which is what :release uses):

https://github.com/bazelbuild/bazel/commit/174ed30685ef3ed30e50e01f547c2fb7a003c12e

+202 -404

0 comment

64 changed files

jparise

pr closed time in 7 days

push eventjparise/PodToBUILD

Jon Parise

commit sha 6a24175625b1a82569bf7cb5219a19124683d0d5

Update React example Podfile deps (#185)

view details

Jon Parise

commit sha f056b6dbdc5ad80680a6b48e9b2471fb19625e77

Merge CI workflows (#187) These were identical, so we don't need separate pipelines for push and pull_request events.

view details

push time in 7 days

delete branch jparise/PodToBUILD

delete branch : ci-merge

delete time in 7 days

push eventpinterest/PodToBUILD

Jon Parise

commit sha f056b6dbdc5ad80680a6b48e9b2471fb19625e77

Merge CI workflows (#187) These were identical, so we don't need separate pipelines for push and pull_request events.

view details

push time in 7 days

PR merged pinterest/PodToBUILD

Merge CI workflows

These were identical, so we don't need separate pipelines for push and pull_request events.

+9 -24

0 comment

2 changed files

jparise

pr closed time in 7 days

push eventjparise/PodToBUILD

Jon Parise

commit sha 04086a97e4927408470c84f6c53a6d41559bf9cb

Merge CI workflows These were identical, so we don't need separate pipelines for push and pull_request events.

view details

push time in 7 days

PR opened pinterest/PodToBUILD

Merge CI workflows

These were identical, so we don't need separate pipelines for push and pull_request events.

+8 -23

0 comment

2 changed files

pr created time in 8 days

create barnchjparise/PodToBUILD

branch : ci-merge

created branch time in 8 days

pull request commentpinterest/PodToBUILD

Create unit test to fix issue #177

Also, it appears that #178 includes a version of this unit test. How does the one here differ?

rquesada

comment created time in 8 days

pull request commentpinterest/PodToBUILD

Fix issue #177

@rquesada I'm not sure why the CI pipeline didn't run for this change. Could you rebase your change on master to see if that kicks it off?

rquesada

comment created time in 8 days

pull request commentpinterest/PodToBUILD

Create unit test to fix issue #177

@rquesada I'm not sure why the CI pipeline didn't run for this change. Could you rebase your change on master to see if that kicks it off?

rquesada

comment created time in 8 days