search

humanmade / hm-base

Standard project layout for Human Made Projects.
81 stars 18 forks source link

Update circle.yml - fix tests #38

Closed roborourke closed 8 years ago

roborourke commented 8 years ago

Sync submodules, currently fails to checkout WP

BronsonQuick commented 8 years ago

@rmccue Got any ideas why CircleCI is throwing this error:

fatal: reference is not a tree: d48ec22bd29af54662d32904978678fbf102705d
Unable to checkout 'd48ec22bd29af54662d32904978678fbf102705d' in submodule path 'wordpress'

git submodule update --init --recursive returned exit code 1

Action failed: git submodule update --init --recursive

I've tried pinning the wordpress submodule to branches and SHA's but CircleCI always throws fatal: reference is not a tree: error

roborourke commented 8 years ago

I was about to had to switch project - you can run the ci build with ssh then poke about the machine. You could check the submodule is pointing to the right place in gitmodules and the checkout itself

Robert O'Rourke Senior WordPress Engineer humanmade

phone: +447816329424 email: rob@hmn.md skype: rob.o.rourke twitr: @sanchothefat https://twitter.com/sanchothefat

BronsonQuick commented 8 years ago

@sanchothefat Cool. Taa. SSH helped! CircleCI has cached .git/modules/wordpress when I manually remove that and run git submodule update --init it checks out. Running CircleCI without caching didn't fix that up. Just gotta read the docs on how to trigger the build manually in SSH and we should be good!

BronsonQuick commented 8 years ago

@sanchothefat 🎉 FYI if this happens again with submodules this is the fix! https://github.com/humanmade/hm-base/pull/38/commits/6cea416f9e13200cfb09cb7b27c7471863d8db7f